https://bugs.contribs.org/show_bug.cgi?id=9033

--- Comment #51 from Charlie Brady 
<[email protected]> ---
> Use of uninitialized value $city in string eq at 
> /usr/share/qpsmtpd/plugins/check_badcountries line 266.

You can do this:

    my $city = $self->qp->connection->notes('geoip_city');
    #warn ("At City (check): $city");
-   if ($city eq '') {
+   unless ($city) {
        $self->log(LOGINFO, "Information: City NA RemoteIP $ip");

or:

    # City Information in case we want to play later
-   my $city = $self->qp->connection->notes('geoip_city');
+   my $city = $self->qp->connection->notes('geoip_city') // '';
    #warn ("At City (check): $city");
    if ($city eq '') {
        $self->log(LOGINFO, "Information: City NA RemoteIP $ip");
    }

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
_______________________________________________
Mail for each SME Contribs bug report
To unsubscribe, e-mail [email protected]
Searchable archive at https://lists.contribs.org/mailman/public/contribteam/

Reply via email to