[Snort-devel] HTTP Reassembly issue PAF enabled
parmendra.pratap at ...398...
Thu Apr 4 08:25:39 EDT 2013
Thanks for a quick reply.
I tried the use case with Snort 18.104.22.168.
Does not make a difference.
Issue is still replicable with the steps outlined in my root email.
This is what I think is going on , based on few tests and source lookups <excuse my newbieness if it reflects anywhere below :) >-
Stream5 reassembly does not tag a packet as complete PDU until it recieves subsequent ack gainst the packet no matter wheter or not the packet actually holds complete PDU (in this case HTTP) or not.
With PAF enabled this prevents the URI Bufs from being created and inspected in HTTP inspect module until the next packet arrives (ie ack against the original packet that contained the HTTP req).
When the HTTP inspect URI Bufs based match fires, with PAF ON, its always(mostly?) when the ack on reverse direction is received.
The spo_alert* modules simply uses the header data provided in the Packet->tcph which holds the header from the current packet ie ack packet from the server ... and hence the incorrect TCP header display with PAF on.
There is no way curently to get the correct TCP headers unless Stream 5 is queried to give the original raw packet <spo_log_tcp_dump.c does that>.
Realworld issue arising from the above is incorrect TCP header data in alerts. TCP dumps are OK though for reason mentioned above.
There seems multiple ways to get around this:
Generate TCP dump on all alerts and assume the alerts will have incorrect TCP headers
Write an alert output plugin that inspects the raw packet for correct TCP headers etc
Add more metadata to Packet struct which can provide the correct TCP headers at least for the last packet that completed the PDU in the alert output plugins.
Last option looks the most organic and least sub optimal one to me.
Given my little experience with snort so far , I wont be surprised if any of the above stated flow is incorrect.
I will more than appreciate if someone can correct me above and enlighten me more about the internals :) .
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Snort-devel