diff options
| author | Evan Huus <eapache@gmail.com> | 2013-03-06 00:39:35 +0000 |
|---|---|---|
| committer | Evan Huus <eapache@gmail.com> | 2013-03-06 00:39:35 +0000 |
| commit | f6156140c37d9256de7a3d395a5a946d21044356 (patch) | |
| tree | 6aaee4becf2851685b723b370e2ea6a87c9bbc39 /epan/reassemble.c | |
| parent | 427790ed8ffd1cf5afa9eb86dad43ae9af21688c (diff) | |
Followup to r48011, answering my own XXX comment:
No, ReportedBoundsError is not the right thing to throw, ReassemblyError is.
That's why I added it in the first place!
svn path=/trunk/; revision=48123
Diffstat (limited to 'epan/reassemble.c')
| -rw-r--r-- | epan/reassemble.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/epan/reassemble.c b/epan/reassemble.c index 3fb07a6d9d..ec351ec700 100644 --- a/epan/reassemble.c +++ b/epan/reassemble.c @@ -748,11 +748,9 @@ fragment_add_work(fragment_data *fd_head, tvbuff_t *tvb, const int offset, /* Otherwise, bail out since we have no idea what to do * with this fragment (and if we keep going we'll run * past the end of a buffer sooner or later). - * - * XXX: Is ReportedBoundsError the right thing to throw? */ g_slice_free(fragment_data, fd); - THROW(ReportedBoundsError); + THROW(ReassemblyError); } } |