diff options
author | Bill Meier <wmeier@newsguy.com> | 2012-07-07 14:21:20 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2012-07-07 14:21:20 +0000 |
commit | 9e7c7e7d10fbe786cda54f96e7cdf89e18e88eee (patch) | |
tree | 00d9bc7641a9bb7a445428eb97e4e3dd65084ba9 /epan/dissectors/packet-dcerpc-nt.c | |
parent | 9315a45deb1957b0e87bde5e0a82ed4f76b87cea (diff) |
From Evan Huus: Fix "Useless null checks in packet-dcerpc*"
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7448
svn path=/trunk/; revision=43599
Diffstat (limited to 'epan/dissectors/packet-dcerpc-nt.c')
-rw-r--r-- | epan/dissectors/packet-dcerpc-nt.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-dcerpc-nt.c b/epan/dissectors/packet-dcerpc-nt.c index 5b768dfdc0..a960a8c2e0 100644 --- a/epan/dissectors/packet-dcerpc-nt.c +++ b/epan/dissectors/packet-dcerpc-nt.c @@ -1312,11 +1312,11 @@ void cb_wstr_postprocess(packet_info *pinfo, proto_tree *tree _U_, proto_item_append_text(item, ": %s", s); item = GET_ITEM_PARENT(item); levels--; - if (item && levels > 0) { + if (levels > 0) { proto_item_append_text(item, ": %s", s); item = GET_ITEM_PARENT(item); levels--; - while (item && levels > 0) { + while (levels > 0) { proto_item_append_text(item, " %s", s); item = GET_ITEM_PARENT(item); levels--; |