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-radius.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-radius.c')
-rw-r--r-- | epan/dissectors/packet-radius.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/epan/dissectors/packet-radius.c b/epan/dissectors/packet-radius.c index 2050c30640..fb14bcb161 100644 --- a/epan/dissectors/packet-radius.c +++ b/epan/dissectors/packet-radius.c @@ -1802,13 +1802,13 @@ dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _ * pointer for the second address argument even * if you do that. */ - conversation = find_conversation(pinfo->fd->num, &pinfo->src, + conversation = find_conversation(pinfo->num, &pinfo->src, &null_address, pinfo->ptype, pinfo->srcport, pinfo->destport, 0); if (conversation == NULL) { /* It's not part of any conversation - create a new one. */ - conversation = conversation_new(pinfo->fd->num, &pinfo->src, + conversation = conversation_new(pinfo->num, &pinfo->src, &null_address, pinfo->ptype, pinfo->srcport, pinfo->destport, 0); } @@ -1825,7 +1825,7 @@ dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _ { /* We've seen a request with this ID, with the same destination, before - but was it *this* request? */ - if (pinfo->fd->num != radius_call->req_num) + if (pinfo->num != radius_call->req_num) { /* No, so it's a duplicate request. Mark it as such. FIXME: This is way too simple, as the request number @@ -1855,7 +1855,7 @@ dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _ new_radius_call_key = wmem_new(wmem_file_scope(), radius_call_info_key); *new_radius_call_key = radius_call_key; radius_call = wmem_new(wmem_file_scope(), radius_call_t); - radius_call->req_num = pinfo->fd->num; + radius_call->req_num = pinfo->num; radius_call->rsp_num = 0; radius_call->ident = rh.rh_ident; radius_call->code = rh.rh_code; @@ -1924,7 +1924,7 @@ dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _ * pointer for the second address argument even * if you do that. */ - conversation = find_conversation(pinfo->fd->num, &null_address, + conversation = find_conversation(pinfo->num, &null_address, &pinfo->dst, pinfo->ptype, pinfo->srcport, pinfo->destport, 0); if (conversation != NULL) { @@ -1987,13 +1987,13 @@ dissect_radius(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data _ /* We have not yet seen a response to that call, so this must be the first response; remember its frame number. */ - radius_call->rsp_num = pinfo->fd->num; + radius_call->rsp_num = pinfo->num; } else { /* We have seen a response to this call - but was it *this* response? (disregard provisional responses) */ - if ( (radius_call->rsp_num != pinfo->fd->num) && (radius_call->rspcode == rh.rh_code) ) + if ( (radius_call->rsp_num != pinfo->num) && (radius_call->rspcode == rh.rh_code) ) { /* No, so it's a duplicate response. Mark it as such. */ rad_info->is_duplicate = TRUE; |