complaint from 4.68.4 (now 4.68.5)

[Jeff A. Earickson]

On Tue, 18 Mar 2008, Julian Field wrote:
> Jeff A. Earickson wrote:
>> Julian,
>>
>> I just installed and tested 4.68.5-1.  Slightly different result, but
>> more or less the same complaint:
>>
>> (65)> ./MailScanner --lint
>> Trying to setlogsock(native)
>> Read 814 hostnames from the phishing whitelist
>> Read 4587 hostnames from the phishing blacklist
>> Config: calling custom init function IPBlock
>> Could not use Custom Function code
>> MailScanner::CustomConfig::InitIPBlock, it could not be "eval"ed. Make
>> sure the module is correct with perl -wc at
>> /opt/MailScanner/lib/MailScanner/Config.pm line 873
> You need to take a look at InitIPBlock in CustomConfig.pm, you have
> something wrong there.

Julian,
Sorry to be dense here, but I don't how it could be me unless I mucked
up the data file (and I don't see anything there).  I haven't
changed anything regarding IPBlock in a while, except for the actual
/etc/MailScanner/IPBlock.conf data file.  Is there any particular
requirement on the white space in this file, eg:

    127.0.0.1[something]100000

Does [something] have to be a tab, or can it be any whitespace? 
My only other twiddles regarding IPBlock are in MailScanner.conf:

    Always Looked Up Last = &IPBlock

And these minor changes to CustomConfig.pm:

    (176)> diff CustomConfig.pm.orig CustomConfig.pm
    777c777
    <   $AccessDB = "/etc/mail/access.db";
    ---
    >   $AccessDB = "/etc/mail/db/access.db";
    1703c1703
    <       $AccessDB = "/etc/mail/access.db";
    ---
    >       $AccessDB = "/etc/mail/db/access.db";

And BTW, shouldn't line 300 of CustomConfig.pm have backticks, eg:

    300c300
    <   $hostname2 = hostname;
    ---
    >   $hostname2 = `hostname`;

to actually execute the hostname and assign it to the variable?

Jeff Earickson
Colby College