From 651ca986ffcf58d503b944c5e9e6a1c16e06cc5e Mon Sep 17 00:00:00 2001
From: Matt Simerson <matt@tnpi.net>
Date: Sun, 22 Apr 2012 17:00:53 -0400
Subject: [PATCH] don't print GeoIP country if not defined

If we don't get a result from the lookup, all we know is that we didn't get a result. Maybe an error, maybe the IP not in the database.
---
 plugins/ident/geoip | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/plugins/ident/geoip b/plugins/ident/geoip
index d7a537c..2e325c7 100644
--- a/plugins/ident/geoip
+++ b/plugins/ident/geoip
@@ -16,14 +16,15 @@ or greylist.
 use Geo::IP;
 
 sub hook_connect {
-  my ($self) = @_;
+    my ($self) = @_;
 
-  my $geoip = Geo::IP->new(GEOIP_STANDARD);
-  my $country = 
-    $geoip->country_code_by_addr( $self->qp->connection->remote_ip );
+    my $geoip = Geo::IP->new(GEOIP_STANDARD);
+    my $country =
+        $geoip->country_code_by_addr( $self->qp->connection->remote_ip )
+            or return (DECLINED);
 
-  $self->qp->connection->notes('geoip_country', $country);
-  $self->log(LOGNOTICE, "GeoIP Country: $country");
+    $self->qp->connection->notes('geoip_country', $country);
+    $self->log(LOGNOTICE, "GeoIP Country: $country");
 
-  return DECLINED;
+    return DECLINED;
 }