Wireshark-commits: [Wireshark-commits] rev 20898: /trunk/ /trunk/epan/dissectors/: packet-rtp.c /tr

Date: Thu, 22 Feb 2007 20:40:22 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=20898

User: etxrab
Date: 2007/02/22 08:40 PM

Log:
 From Richard van der Hoff:
 Along with this bug, identified by Mark, there is another problem, in that one of the chunks of my earlier patch seemed to get missed off when Anders committed it. This won't break anything yet, as the H.223-over-RTP dissection hasn't landed on trunk yet, but it will cause all sorts of nasties when it does.
 
 Here is a new patch, against current trunk, which should fix Mark's bug, my bug, and a comment typo.
 
 And a patch wich improves the general robustness of the h.223 dissector (making it less likely to crash on malformed data).
 
 Hopefully this also fixes a bug raised by Fabio Sguanci a few weeks ago. 
 Fabio: I think a better way to fix the problem is to stop the dissector crashing when it finds a malformed PDU, so that it just treats the first pdu as malformed; there is then no need to special-case it.

Directory: /trunk/epan/dissectors/
  Changes    Path            Action
  +5 -4      packet-rtp.c    Modified

Directory: /trunk/plugins/h223/
  Changes    Path             Action
  +295 -222  packet-h223.c    Modified