[Greylist-users] error messages...

[Allan E Johannesen]

I have seen errors of non numeric arguments in the perl script.  The line is:

  # Only if we have some rowids, do we update the count of passed messages
  if ($rowids > 0) {

Should this be changed to be a loop through the rowids if multiple rows are
returned?  Does it actually perform the if, given the error?  Since there is a
note about incremented pass counts, is this error intended to occur?

Below is the error, and a select from the table after the update:


=== 2003-12-14 21:20:03 ===
Stored Sender: <wseagrave at charter.net>
Passed Recipient: <wseagrav at WPI.EDU>
  Relay: mxsf21.cluster1.charter.net [209.225.28.221] - If_Addr: 130.215.36.202
  RelayIP: 209.225.28.221 - RelayName: mxsf21.cluster1.charter.net -
RelayIdent:  - PossiblyForged: 0
  From: wseagrave at charter.net - To: wseagrav at WPI.EDU
  InMailer: relay - OutMailer: relay - QueueID: hBF2K2Rl015861
  Email is known and block has expired.  Passing the mail.  rowid: 90281
Argument "90284,90283" isn't numeric in numeric gt (>) at
/usr/local/sbin/relaydelay.pl line 894.
  IN EOM CALLBACK - PrivData:
90284,90283,90281^@<wseagrave at charter.net>^@<wseagrav at WPI.EDU>
Argument "90284,90283,90281" isn't numeric in numeric gt (>) at
/usr/local/sbin/relaydelay.pl line 381.
  * Mail successfully processed.  Incremented passed count on rowid 90284.
  * Mail successfully processed.  Incremented passed count on rowid 90283.
  * Mail successfully processed.  Incremented passed count on rowid 90281.


mysql> select * from relaytofrom where mail_from = '<wseagrave at charter.net>';
+-------+----------------+-------------------------+--------------------+---------------------+---------------------+---------------+--------------+---------------+-------------+---------------------+----------------+
| id    | relay_ip       | mail_from               | rcpt_to            | block_expires       | record_expires      | blocked_count | passed_count | aborted_count | origin_type | create_time         | last_update    |
+-------+----------------+-------------------------+--------------------+---------------------+---------------------+---------------+--------------+---------------+-------------+---------------------+----------------+
| 90281 | 209.225.28.221 | <wseagrave at charter.net> | <wseagrav at WPI.EDU> | 2003-12-14 21:00:26 | 2004-01-19 21:20:03 |            16 |            1 |             0 | AUTO        | 2003-12-14 20:02:26 | 20031214212003 |
| 90283 | 209.225.28.221 | <wseagrave at charter.net> | <shook at WPI.EDU>    | 2003-12-14 21:00:26 | 2004-01-19 21:20:03 |            16 |            1 |             0 | AUTO        | 2003-12-14 20:02:26 | 20031214212003 |
| 90284 | 209.225.28.221 | <wseagrave at charter.net> | <gboyer at WPI.EDU>   | 2003-12-14 21:00:26 | 2004-01-19 21:20:03 |            16 |            1 |             0 | AUTO        | 2003-12-14 20:02:26 | 20031214212003 |
+-------+----------------+-------------------------+--------------------+---------------------+---------------------+---------------+--------------+---------------+-------------+---------------------+----------------+
3 rows in set (0.00 sec)