search

benningm / mtpolicyd

a modular policy daemon for postfix
21 stars 3 forks source link

Incorrect assignment of $count variable in is_autowl and add_autowl functions of Greylist.pm #37

Open alescano opened 2 years ago

alescano commented 2 years ago

Hi @benningm,

I think in these functions return value of do_cached function is being not properly assigned because of the missing parenthesis around $count ( lines 290 and 313 ). I noticed because autowl count in the database was never being incremented.

https://github.com/benningm/mtpolicyd/blob/23a4c606ce978eb0dce60f46e9c9ccf81005a582/lib/Mail/MtPolicyd/Plugin/Greylist.pm#L286-L326