diff options
author | Evan Huus <eapache@gmail.com> | 2014-11-05 23:43:20 -0500 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2014-11-06 04:45:16 +0000 |
commit | c321dc3f2e000213c8c626052e304e2a510c94a6 (patch) | |
tree | 0c0ad86fbc7be9fc74aa80a5f202b9445abcfc62 /epan | |
parent | d0c031ff19167e7c99f46b145c2eab890971e790 (diff) |
rpc: fix regression in gbfc5483174d
value can be NULL, in which case memduping it will crash
Bug: 10667
Change-Id: I8950fae94d80a2f06f24a785532536ad73f1855d
Reviewed-on: https://code.wireshark.org/review/5152
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-rpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-rpc.c b/epan/dissectors/packet-rpc.c index 51a0e48a6e..c2ab8dde2b 100644 --- a/epan/dissectors/packet-rpc.c +++ b/epan/dissectors/packet-rpc.c @@ -2334,7 +2334,7 @@ dissect_rpc_message(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, rpc_call->flavor = flavor; rpc_call->gss_proc = gss_proc; rpc_call->gss_svc = gss_svc; - rpc_call->proc_info = (rpc_proc_info_value*)wmem_memdup(wmem_file_scope(), value, sizeof(*value)); + rpc_call->proc_info = value ? (rpc_proc_info_value*)wmem_memdup(wmem_file_scope(), value, sizeof(*value)) : NULL; rpc_call->req_time = pinfo->fd->abs_ts; /* store it */ |