diff options
author | Guy Harris <guy@alum.mit.edu> | 2011-02-17 04:35:12 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2011-02-17 04:35:12 +0000 |
commit | a12338b76ab50eb2b0bebff65c7540f8389658e2 (patch) | |
tree | 10602dd528bd340655e02730be893ee51907e138 /epan/dissectors/packet-isakmp.c | |
parent | 6c6f3528f1ce4d6c2795e27ddaea015d68d97c2b (diff) |
Use "XXX != NULL" rather than "XXX" to test for a null pointer; either
I'm missing something or the MSVC++ code analyzer doesn't realize that
in
if (XXX)
dereference XXX
will not dereference XXX if it's null - maybe "if (XXX != NULL)" will do
the trick (if so, the code analyzer is buggy, because "if (XXX !=
NULL)", "if (XXX != 0)", and "if (XXX)" mean the exact same thing if XXX
is a pointer-valued expression, really, truly, even if a null pointer
isn't represented as all zero bits or if it's wider than an int).
Clean up indentation.
svn path=/trunk/; revision=35973
Diffstat (limited to 'epan/dissectors/packet-isakmp.c')
0 files changed, 0 insertions, 0 deletions