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

LAPSat: remove a useless assert 

If the length is bigger than the captured length, an excpetion will be thrown anyway

Bug: 13554
Change-Id: Icf0f2b049847ec945efdf42815b49c4e6040e19d
Reviewed-on: https://code.wireshark.org/review/20936
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
This commit is contained in:
Pascal Quantin 2017-04-05 21:33:15 +02:00 committed by Anders Broman
parent f53333f433
commit 462fba965a
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
epan/dissectors/packet-lapsat.c
View File

@ -490,8 +490,6 @@ dissect_lapsat(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* dissec
if (!plen)
return 3; /* No point in doing more if there is no payload */
DISSECTOR_ASSERT((plen + hlen) <= tvb_captured_length(tvb));
if ((plen + hlen) == tvb_captured_length(tvb)) {
/* Need to integrate the last nibble */
guint8 *data = (guint8 *)tvb_memdup(pinfo->pool, tvb, hlen, plen);