osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

bssmap_msg_fcn[] is a dense array, not a sparse array, so *every* entry 

must be filled in - even if we don't happen to have dissectors for
particular message types.  Just put NULL in there, so we don't index
past the end of the array, grab a random location in memory's contents
as a function pointer, and crash when we call through that pointer.

svn path=/trunk/; revision=46747
This commit is contained in:
Guy Harris 2012-12-25 23:14:05 +00:00
parent 8432edcc72
commit 8ede6b7dc0
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
epan/dissectors/packet-gsm_a_bssmap.c
View File

@ -6864,12 +6864,13 @@ static void (*bssmap_msg_fcn[])(tvbuff_t *tvb, proto_tree *tree, packet_info *pi
bssmap_int_ho_req_rej, /* 0x71 Internal Handover Required Reject */
bssmap_int_ho_cmd, /* 0x72 Internal Handover Command */
bssmap_int_ho_enq, /* 0x73 Internal Handover Enquiry */
/* 0x74 LCLS-Connect-Control */
/* 0x75 LCLS-Connect-Control-Ack */
/* 0x76 LCLS-Notification */
/* 0x77 Unallocated */
/* 0x78 Reroute Command */
/* 0x79 Reroute Complete */
NULL, /* 0x74 LCLS-Connect-Control */
NULL, /* 0x75 LCLS-Connect-Control-Ack */
NULL, /* 0x76 LCLS-Notification */
NULL, /* 0x77 Unallocated */
NULL, /* 0x78 Reroute Command */
NULL, /* 0x79 Reroute Complete */
NULL, /* NONE */
};