diff options
| author | Bryon Gloden, CISSP® <cissp@bryongloden.com> | 2016-08-02 17:49:49 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-08-02 17:49:49 -0400 |
| commit | 8086172ad1e4d4e138e2ba80056b8b5835d95a60 (patch) | |
| tree | 9f088fcdf97e31b8807f74fb250086d335e1fe2f /contrib/exim | |
| parent | 0ef106e8a3a39c3dbd8e2cc5f24fff7e0e23ac40 (diff) | |
| download | rspamd-8086172ad1e4d4e138e2ba80056b8b5835d95a60.tar.gz rspamd-8086172ad1e4d4e138e2ba80056b8b5835d95a60.zip | |
Update local_scan.c
[../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
Diffstat (limited to 'contrib/exim')
| -rw-r--r-- | contrib/exim/local_scan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/exim/local_scan.c b/contrib/exim/local_scan.c index 20d3ba246..3277bd879 100644 --- a/contrib/exim/local_scan.c +++ b/contrib/exim/local_scan.c @@ -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; |