[PATCH] SAVAPI antivirus: Add missing terminal status codes

Thomas Jarosch thomas.jarosch at intra2net.com
Fri Nov 10 10:17:30 CET 2017


Add terminal status code 350 (scan error) and 404 (too many clients connected).

As soon as there's a terminal status code, we should skip further processing.
Otherwise amavisd might process bogus replies from savapi as valid data.

Signed-off-by: Thomas Jarosch <thomas.jarosch at intra2net.com>
---
 amavisd | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/amavisd b/amavisd
index 7f93194..69605ad 100755
--- a/amavisd
+++ b/amavisd
@@ -28379,7 +28379,7 @@ sub avira_savapi_internal {
       } else {
         $output .= $ln  if length($output) < 10000;  # sanity limit
       }
-      last if $ln =~ /^([0125-9]\d\d|300|319).*\012/;  # terminal status
+      last if $ln =~ /^([0125-9]\d\d|300|319|350|404).*\012/;  # terminal status
     # last if $ln =~ !/^(310|420|421|422|430).*\012/;  # nonterminal status
     }
   }
-- 
2.13.6



More information about the amavis-users mailing list