diff options
author | Guy Harris <guy@alum.mit.edu> | 2014-10-01 13:12:13 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2014-10-01 13:12:13 -0700 |
commit | 214eb814eeced4f3ffa3c84455f0b168c756e57e (patch) | |
tree | b526e8383f7dae4f50356102335fc2687ff76be5 /print-pim.c | |
parent | 04a0674702b80174582d230ce9374bf06650789e (diff) | |
download | tcpdump-214eb814eeced4f3ffa3c84455f0b168c756e57e.tar.gz |
Add some more parentheses, Just In Case.
I'm not sure whether
Performing a byte swapping operation on "p" implies that it came
from an external source, and is therefore tainted.
from Coverity means that it thinks we're byte-swapping the pointer
*itself*, or that we're byte-swapping what it points to, but, just in
case it's the former, let's try throwing some more parentheses in.
(If it's the latter, well, yes, it's packet data, so it comes from an
external source, but Coverity didn't seem to point out any place where
we were using the data it points to without checking its value in cases
where we have to.)
Diffstat (limited to 'print-pim.c')
0 files changed, 0 insertions, 0 deletions