Wireshark-commits: [Wireshark-commits] rev 20888: /trunk/epan/ /trunk/epan/: reassemble.c reassembl

Date: Wed, 21 Feb 2007 06:19:04 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=20888

User: etxrab
Date: 2007/02/21 06:19 AM

Log:
 From Richard van der Hoff:
 01_reassemble_test.patch
 ------------------------
 I didn't want to do anything without some unit tests, so here they are. 
 This allows a standalone binary, epan/reassemble_test, to be built; this can be run from the commandline and should end up printing out "success" 
 if all goes well.
 NOTE the changes to makefile.am NOT checked in currently.
 
 Incidentally: is it possible to get the buildbot to run things like this, exntest and tvbtest?
 
 02_reassemble_refactor.patch
 ----------------------------
 fragment_add_seq, fragment_add_dcerpc_dg and fragment_add_seq_check_work were all pretty much carbon-copies of each other. This patch factors out the common parts of the routines into a new routine, fragment_add_seq_key().
 
 03_reassemble_partial_reassembly.patch
 ---------------------------------------
 This makes fragment_set_partial_reassembly() work for datagrams assembled with fragment_add_seq(). The patch itself is actually quite small, but it adds another unit test which is reasonably lengthy.

Directory: /trunk/epan/
  Changes    Path                 Action
  +230 -214  reassemble.c         Modified
  +79 -34    reassemble.h         Modified
  +1057 -0   reassemble_test.c    Added