diff options
| author | Dario Lombardo <lomato@gmail.com> | 2019-12-01 21:55:11 +0100 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2019-12-02 00:33:45 +0000 |
| commit | 8dcd1e2bcd1037f6ee2f7e5ec6e853006c4f2634 (patch) | |
| tree | 230b5f2eb6fef6737a440063af02a246a6d15c84 /epan/dissectors/packet-kafka.c | |
| parent | e06969455bf561766601cfc3beeaa7fb90ecf44a (diff) | |
kafka: don't use an empty tvb list.
Bug: 16242
Change-Id: I1a7cfa504d46cab681c7803227102cafcda519fa
Reviewed-on: https://code.wireshark.org/review/35277
Petri-Dish: Dario Lombardo <lomato@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-kafka.c')
| -rw-r--r-- | epan/dissectors/packet-kafka.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/epan/dissectors/packet-kafka.c b/epan/dissectors/packet-kafka.c index d6a84b07a7..0beb219692 100644 --- a/epan/dissectors/packet-kafka.c +++ b/epan/dissectors/packet-kafka.c @@ -1357,7 +1357,7 @@ decompress_lz4(tvbuff_t *tvb, packet_info *pinfo, int offset, guint32 length, tv do { src_size = length - src_offset; // set the number of available octets if (src_size == 0) { - break; + goto end; } decompressed_buffer = (guchar*)wmem_alloc(pinfo->pool, dst_size); rc = LZ4F_decompress(lz4_ctxt, decompressed_buffer, &dst_size, @@ -1366,8 +1366,7 @@ decompress_lz4(tvbuff_t *tvb, packet_info *pinfo, int offset, guint32 length, tv goto end; } if (dst_size == 0) { - // XXX - can this happen? - break; + goto end; } tvb_composite_append(composite_tvb, tvb_new_child_real_data(tvb, (guint8*)decompressed_buffer, (guint)dst_size, (gint)dst_size)); |