diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-01-23 19:40:51 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-01-24 03:41:28 +0000 |
commit | bc5a0374bfd162d08834f5f7503bebd33d8ec943 (patch) | |
tree | 6d5be93a3e35c6eb144ce6d2b1d95650b5cbbd86 /epan/dissectors/packet-mbim.c | |
parent | baea677290f84d4e30e86194c79bafef0fdc1ad2 (diff) |
Add the packet number to the packet_info structure, and use it.
That removes most of the uses of the frame number field in the
frame_data structure.
Change-Id: Ie22e4533e87f8360d7c0a61ca6ffb796cc233f22
Reviewed-on: https://code.wireshark.org/review/13509
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan/dissectors/packet-mbim.c')
-rw-r--r-- | epan/dissectors/packet-mbim.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/epan/dissectors/packet-mbim.c b/epan/dissectors/packet-mbim.c index 7c988a2f7e..5d72ec3b27 100644 --- a/epan/dissectors/packet-mbim.c +++ b/epan/dissectors/packet-mbim.c @@ -3802,7 +3802,7 @@ dissect_mbim_control(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void * mbim_conv->open = wmem_tree_new(wmem_file_scope()); mbim_conv->cellular_class = 0; mbim_conv->open_count = 0; - wmem_tree_insert32(mbim_conv->open, PINFO_FD_NUM(pinfo), GUINT_TO_POINTER(mbim_conv->open_count)); + wmem_tree_insert32(mbim_conv->open, pinfo->num, GUINT_TO_POINTER(mbim_conv->open_count)); conversation_add_proto_data(conversation, proto_mbim, mbim_conv); } @@ -3827,7 +3827,7 @@ dissect_mbim_control(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void * if (!PINFO_FD_VISITED(pinfo)) { mbim_conv->open_count++; - wmem_tree_insert32(mbim_conv->open, PINFO_FD_NUM(pinfo), GUINT_TO_POINTER(mbim_conv->open_count)); + wmem_tree_insert32(mbim_conv->open, pinfo->num, GUINT_TO_POINTER(mbim_conv->open_count)); } max_ctrl_transfer = tvb_get_letohl(tvb, offset); if (max_ctrl_transfer == 8) { @@ -3873,13 +3873,13 @@ dissect_mbim_control(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void * frag_tvb = tvb; } - open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, PINFO_FD_NUM(pinfo))); + open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, pinfo->num)); trans_id_key = ((guint64)open_count << 32) | trans_id; if (!PINFO_FD_VISITED(pinfo)) { p_trans_id_key = wmem_new(wmem_file_scope(), guint64); *p_trans_id_key = trans_id_key; mbim_info = wmem_new(wmem_file_scope(), struct mbim_info); - mbim_info->req_frame = PINFO_FD_NUM(pinfo); + mbim_info->req_frame = pinfo->num; mbim_info->resp_frame = 0; wmem_map_insert(mbim_conv->trans, p_trans_id_key, mbim_info); } else { @@ -4334,12 +4334,12 @@ dissect_mbim_control(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void * } break; case MBIM_FUNCTION_ERROR_MSG: - open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, PINFO_FD_NUM(pinfo))); + open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, pinfo->num)); trans_id_key = ((guint64)open_count << 32) | trans_id; mbim_info = (struct mbim_info *)wmem_map_lookup(mbim_conv->trans, &trans_id_key); if (!PINFO_FD_VISITED(pinfo)) { if (mbim_info) { - mbim_info->resp_frame = PINFO_FD_NUM(pinfo); + mbim_info->resp_frame = pinfo->num; } } else { if (mbim_info && mbim_info->req_frame) { @@ -4390,12 +4390,12 @@ dissect_mbim_control(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void * } if (msg_type == MBIM_COMMAND_DONE) { - open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, PINFO_FD_NUM(pinfo))); + open_count = GPOINTER_TO_UINT(wmem_tree_lookup32_le(mbim_conv->open, pinfo->num)); trans_id_key = ((guint64)open_count << 32) | trans_id; mbim_info = (struct mbim_info *)wmem_map_lookup(mbim_conv->trans, &trans_id_key); if (!PINFO_FD_VISITED(pinfo)) { if (mbim_info) { - mbim_info->resp_frame = PINFO_FD_NUM(pinfo); + mbim_info->resp_frame = pinfo->num; } } else { if (mbim_info && mbim_info->req_frame) { |