diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-12-13 02:35:26 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2013-12-13 02:35:26 +0000 |
| commit | 18398c97496ebd0c6699b80357c8853b815d1cb4 (patch) | |
| tree | 47047e5a0a162243a2613d93a53dc7645d3f3a7a /epan/dissectors/packet-lapd.c | |
| parent | e45b95fa02e62190b85750ffdba872a3f4d8f027 (diff) | |
Fix Visual Studio Code Analysis warning C6204: Possible buffer overrun in call to 'memcpy': use of unchecked parameter 'data_len'
svn path=/trunk/; revision=54010
Diffstat (limited to 'epan/dissectors/packet-lapd.c')
| -rw-r--r-- | epan/dissectors/packet-lapd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-lapd.c b/epan/dissectors/packet-lapd.c index 5e2d6f659e..425f070a75 100644 --- a/epan/dissectors/packet-lapd.c +++ b/epan/dissectors/packet-lapd.c @@ -183,8 +183,8 @@ fill_lapd_byte_state(lapd_byte_state_t *ptr, enum lapd_bitstream_states state, c ptr->bit_offset = bit_offset; ptr->ones = ones; - memcpy(ptr->data, data, data_len); - ptr->data_len = data_len; + ptr->data_len = MIN(sizeof(ptr->data), data_len); + memcpy(ptr->data, data, ptr->data_len); } typedef struct lapd_convo_data { |