On Wed, 2008-11-12 at 14:48 -0500, Sebastien Roy wrote:
> I've found the problem and have come up with a fix which I've tested.
> The problem was with the protocol version matching in the ipnet header.
> The protocol field isn't a 2-byte field at offset 0, but a 1 byte field
> at offset 1.
> 
> http://zhadum.east/ws/seb/seb-onfix/webrev/
> 
> I'd like to get both of these fixes RTI'ed today if possible.  Meem, can
> you look this over, I believe that Phil is offline at this point?  Phil
> has already reviewed the fix to 6770479 in snoop_pf.c, so what's left to
> be reviewed is the fix to 6770744 in snoop_filter.c and snoop_ether.c.

In case you're curious, the test results are here:

file:///home/seb/work/bugs/6770479/snoop-testing.txt

-Seb



Reply via email to