diff options
author | Gerald Combs <gerald@wireshark.org> | 2011-04-16 21:46:15 +0000 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2011-04-16 21:46:15 +0000 |
commit | 43b089db8797dbf31306008aca1fee45e7343a25 (patch) | |
tree | 84c707caa6a63abe9994288893ea52e16f07aa6e /epan/dissectors/packet-dcm.c | |
parent | ed25109219f48797edae4d188a4364cd1f22c543 (diff) |
Fix Coverity CIDs 603 & 604.
svn path=/trunk/; revision=36672
Diffstat (limited to 'epan/dissectors/packet-dcm.c')
-rw-r--r-- | epan/dissectors/packet-dcm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-dcm.c b/epan/dissectors/packet-dcm.c index eddcc682c4..1ea7ed100f 100644 --- a/epan/dissectors/packet-dcm.c +++ b/epan/dissectors/packet-dcm.c @@ -5220,7 +5220,7 @@ dissect_dcm_pdv_header(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, so we should always hit this for a correct capture */ - if (pctx && pctx->abss_desc && g_str_has_suffix(pctx->abss_desc, "Storage")) { + if (pctx->abss_desc && g_str_has_suffix(pctx->abss_desc, "Storage")) { /* Should be done far more intelligent, e.g. does not catch the (Retired) ones */ if (flags == 0) { g_snprintf(desc_header, MAX_BUF_LEN, "%s Fragment", pctx->abss_desc); @@ -6263,7 +6263,7 @@ dissect_dcm_pdv_fragmented(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, if (head && head->reassembled_in != pinfo->fd->num) { - if (pdv && pdv->desc) { + if (pdv->desc) { /* We know the presentation context already */ g_snprintf(*pdv_description, MAX_BUF_LEN, "%s (reassembled in #%u)", pdv->desc, head->reassembled_in); } |