Ethereal-dev: [Ethereal-dev] Patch for Bug #817

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: Mike Duigou <ethereal@xxxxxxxxxx>
Date: Wed, 22 Mar 2006 15:21:39 -0800
I have improved the heuristics and the display tree building code in
dissect_jxta_udp() and dissect_jxta_stream() to avoid this problem.

Enclosed is a patch which corrects the problem.
There remains one point of confusion for me. Heuristic dissectors are supposed to return TRUE or FALSE. Assuming that the underlying dissection routine is a new style dissector returning "int" it is obvious that : 
	new dissector result > 0 --> heuristic returns TRUE
	new dissector result == 0 --> heuristic returns FALSE
but what about :
	new dissector result < 0 --> heuristic returns ?????
If I suspect that a packet may contain JXTA data but need more data to make a determination what should my heuristic dissector return? 

Mike

Attachment: jxta_17700.diff.gz
Description: GNU Zip compressed data