diff options
author | Evan Huus <eapache@gmail.com> | 2014-04-23 16:10:51 -0400 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2014-04-24 02:16:46 +0000 |
commit | bf1f30b63adac4e1a39a7f0af601789c7755bd64 (patch) | |
tree | 62c5816e33766eced5b280b297466db41242e097 /epan/dissectors/packet-jxta.c | |
parent | fbc522633ead182536e722ad3e29bc077ca3ffa2 (diff) |
JXTA: ask for one more segment, not one more byte
Otherwise we end up doing reassembly of long messages one painful byte at a time
even when all of those bytes are in the same TCP payload. This results in
ridiculous memory usage.
Change-Id: Ie28d5ade1fec54e6ebc225341582270651d7371c
Closes-Bug: 10018
Reviewed-on: https://code.wireshark.org/review/1312
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-jxta.c')
-rw-r--r-- | epan/dissectors/packet-jxta.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-jxta.c b/epan/dissectors/packet-jxta.c index 5a44ac5a61..2ee53e241a 100644 --- a/epan/dissectors/packet-jxta.c +++ b/epan/dissectors/packet-jxta.c @@ -780,7 +780,7 @@ static int dissect_jxta_welcome(tvbuff_t * tvb, packet_info * pinfo, proto_tree return 0; } else { /* ask for more bytes */ - return -1; + return -DESEGMENT_ONE_MORE_SEGMENT; } } |