[Greylist-users] Argument "xxxx" isn't numeric errors/warnings

[Brian Ross]

What's the status on the CVS version being officially released?


At 02:38 PM 4/20/2006, Evan Harris wrote:

>This has been fixed in the version in CVS.  If you want to patch manually,
>it was also discussed on the mailing list a while ago.
>
>Evan
>
>
>On Tue, 18 Apr 2006, Dennis Wynne wrote:
>
> > Noticed a bunch of these in the relaydelay log.
> >
> >
> > Argument "15449,15450,8623" isn't numeric in numeric gt (>) at
> > /etc/mail/relaydelay.pl line 385.
> >
> > Argument "15449,15450" isn't numeric in numeric gt (>) at
> > /etc/mail/relaydelay.pl line 895.
> >
> > Argument "15540,15541" isn't numeric in numeric gt (>) at
> > /etc/mail/relaydelay.pl line 895.
> >
> >
> > Line 895 is the if statement here:
> >
> >    if ($rowids > 0) {
> >       $rowids .= ",$rowid";
> >    } else {
> >      $rowids = $rowid;
> >    }
> >
> > Line 385 is the if statement here:
> >
> >  # Only if we have some rowids, do we update the count of passed messages
> >  if ($rowids > 0) {
> >    # split up the rowids and update each in turn
> >    my @rowids = split(",", $rowids);
> >
> >
> > These appear to be e-mails with multiple mail_to's as each row id shown in
> > the message group has the same IP and mail_from, but different rcpt_to.
> >
> > Do I have a problem?  It seems to be working and passes the mail and
> > increments the counter on these rows.
> >
> > If $rowid contains "15449,15450" which based on the comments in the code it
> > COULD then comparing it to a number (0) would always cause the
> > error/warning.
> >
> >
> >  # Save our private data (since it isn't available in the same form later)
> >  #   The format is a comma seperated list of rowids (or zero if none),
> >  #     followed by the envelope sender followed by the current envelope
> >  #     recipient (or empty string if none) seperated by nulls
> >
> > Seems like it would be better to check it as a string:  if($rowids ne "0")
> >
> > Right?
> >
> > Thanks,
> > Dennis
> >
> >
> >
> > _______________________________________________
> > Greylist-users mailing list
> > Greylist-users at lists.puremagic.com
> > http://lists.puremagic.com/cgi-bin/mailman/listinfo/greylist-users
> >
>_______________________________________________
>Greylist-users mailing list
>Greylist-users at lists.puremagic.com
>http://lists.puremagic.com/cgi-bin/mailman/listinfo/greylist-users