diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-09-16 13:23:34 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-09-16 20:24:38 +0000 |
commit | 4b8504f39a07e19f40b9df1548f8400ef6ae61a8 (patch) | |
tree | cdee2d40e385441256903eabe7544da4ae7a3df5 /wiretap/vwr.c | |
parent | b93ef37da21699336273e846651dd4e7c499f4dc (diff) |
Inquire what frame_size < msdu_length means.
Change-Id: Ie618ed61fb098bfe55529e31e9cc3f6bfe7d4ac0
Reviewed-on: https://code.wireshark.org/review/23575
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/vwr.c')
-rw-r--r-- | wiretap/vwr.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/wiretap/vwr.c b/wiretap/vwr.c index 9312a283de..b8be2f25d2 100644 --- a/wiretap/vwr.c +++ b/wiretap/vwr.c @@ -1990,8 +1990,13 @@ static gboolean vwr_read_s3_W_rec(vwr_t *vwr, struct wtap_pkthdr *phdr, frame_size = rec_size - (stats_offset + vwr->MPDU_OFF + vVW510021_W_STATS_TRAILER_LEN); if (frame_size > ((int) msdu_length)) actual_octets = msdu_length; - else + else { + /* + * XXX - does this mean "the packet was cut short during + * capture" or "this is a malformed record"? + */ actual_octets = frame_size; + } } else { |