On 5/29/07, <b class="gmail_sendername"><a href="mailto:jmzhou.ml@gmail.com">jmzhou.ml@gmail.com</a></b> <<a href="mailto:jmzhou.ml@gmail.com">jmzhou.ml@gmail.com</a>> wrote:<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Some comments about the patch set<br><br>Patch 4: I can confirm that this is a bug. I had the same problem and<br>same fix at my side.<br><br>Patch 5: I think the new call (DoGenParseCode) in pac_type.cc should be<br>replaced by GenParseCode3. Imagine that a record field (identifier f1)
<br>is empty type and there are other fields (identifier f2, f3, ...) after<br>field f1. If you call DoGenParseCode, the identifier f1 will not be<br>evaluated like in GenParseCode3 (pac_type.cc:754-755). This will result
<br>in re-entrance of RecordDataField::GenParseCode (pac_record.cc:420) because<br>when f2->prev()-GenParseCode (pac_record.cc:428) is called, env->Evaluated(<br>id()) will return false. This is an infinite loop - till at some point of
<br>binpac there will be an assertion failure.</blockquote><div><br>I think I can see your point and also that it might generally be more
safe to call GenParseCode3 instead (I attached the updated patch file).<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Patch 6: I think there is another issue here: if some buffering request
<br>has been sent to the flow buffer, and the flow buffer is not ready with<br>partial data, we will end up with entering the loop - a waste of CPU<br>cycles.</blockquote><div><br>Maybe you are right, but what would you suggest as change? Do you want
to check whether the buffer is ready before entering the loop? But then
it has to be ensured that the buffer is properly initialized in any
case. At the moment I cannot see all the consequences of such a
change. And do you think that the impact on performance is really
relevant?
<br><br>Thanks for the comments,<br>Tobias </div><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">_______________________________________________
<br>Bro mailing list<br><a href="mailto:bro@bro-ids.org">bro@bro-ids.org</a><br><a href="http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/bro">http://mailman.ICSI.Berkeley.EDU/mailman/listinfo/bro</a><br><br></blockquote>
</div><br>