diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-09-15 15:20:26 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-09-15 22:21:17 +0000 |
commit | 7a7d162a494e3ccf15f58f1d710dcf645cfa02b1 (patch) | |
tree | ac2d153f0e913a8d8f0be597610c7940d10c2dbf /wiretap/ascend.y | |
parent | acb68ae1c461f8608389b130c8e452bda04d40da (diff) |
Don't pick up junk from an unset error-number variable.
Keep the actual error code and pointer-to-error-string in the scanner
state, rather than pointers to the variables passed in to us.
Initialize them to 0 and NULL, respectively.
That way, when the actual scanner routine returns, we don't check for an
error by looking at the error variable pointed to by our argument, which
might not have been set by the scanner and might have stack junk in it,
we look at a structure member we set to 0 before the scan.
Bug: 12903
Change-Id: I5a382da569a226e60c3c2a47f3a1515b0490c31d
Reviewed-on: https://code.wireshark.org/review/17716
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/ascend.y')
0 files changed, 0 insertions, 0 deletions