diff options
| author | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-08-01 01:50:27 +0000 |
|---|---|---|
| committer | Chris Maynard <Christopher.Maynard@GTECH.COM> | 2012-08-01 01:50:27 +0000 |
| commit | 07503d2af52fc32b88cfd7ab57edee093cacd6e2 (patch) | |
| tree | a1152eccc6ae30e273d6fbb2dea12fb1e21a7184 /epan/dissectors/packet-q931.c | |
| parent | 0c3fd84ef5072a5ea60eccf780f06ef56f5a3f6a (diff) | |
Fix the 2nd part of Coverity CID 280361 (formerly 361): "Logically dead code" that mmartin seems to have unintentionally missed in r36256.
svn path=/trunk/; revision=44177
Diffstat (limited to 'epan/dissectors/packet-q931.c')
| -rw-r--r-- | epan/dissectors/packet-q931.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-q931.c b/epan/dissectors/packet-q931.c index 7518fde4a2..d181c0a2b2 100644 --- a/epan/dissectors/packet-q931.c +++ b/epan/dissectors/packet-q931.c @@ -1060,7 +1060,7 @@ l2_done: add_l3_info = (octet & 0x0F) << 4; if (octet & Q931_IE_VL_EXTENSION) goto l3_done; - if (len == 0) + if (len < 2) return; octet = tvb_get_guint8(tvb, offset + 1); add_l3_info |= (octet & 0x0F); |