]> source.dussan.org Git - rspamd.git/commitdiff
Update local_scan.c 796/head
authorBryon Gloden, CISSPĀ® <cissp@bryongloden.com>
Tue, 2 Aug 2016 21:49:49 +0000 (17:49 -0400)
committerGitHub <noreply@github.com>
Tue, 2 Aug 2016 21:49:49 +0000 (17:49 -0400)
[../rspamd-master/contrib/exim/local_scan.c:630]: (error) Uninitialized variable: rej

It's always good to initialize integer variables, at least to 0, because if you try to retrieve its value before it gets assigned any actual (non-garbage) value, then it results in undefined behavior.

Found by https://github.com/bryongloden/cppcheck

contrib/exim/local_scan.c

index 20d3ba246db6950466e5d34d0bb89b9a866596f9..3277bd87948aa64522cf535c532ba716fff172f0 100644 (file)
@@ -485,7 +485,7 @@ repeat_read:
 int WaitForScanResult (uschar **resStr)
 {
     int Len, i;
-    int rej, result = LOCAL_SCAN_ACCEPT, answer_size, spm = 0, code = 0, ns = 0, smb = 0, urf = 0;
+    int rej = 0, result = LOCAL_SCAN_ACCEPT, answer_size, spm = 0, code = 0, ns = 0, smb = 0, urf = 0;
     char *strP, *tok, *tmp;
     char *hdr = NULL, *hdrv = NULL, *spmStr = NULL, *symbols=NULL, *urls=NULL;
     char answ [4096], state[6], metric[128], back;