[Bro] Small bug in TCP_Rewriter

Vern Paxson vern at icir.org
Tue Aug 23 11:01:18 PDT 2005


[catching up on some old threads - apologies for the delay]

> perhaps ..
> 
>          if(!next_packet->AppendData(data, left))
>          {     ASSERT(0);  }

Thanks, that looks like the correct fix.  I've added this.

By the way, does anyone know whether ASSERT() is guaranteed to be equivalent
to a statement, so that the {}'s above can be safely omitted?

> Plz. let me know if this isn't the appropriate forum for submitting bugs.

This is a fine forum for doing so (with again my apologies for the delay
in following up).

		Vern



More information about the Bro mailing list