23 Oct
2013
23 Oct
'13
12:58 a.m.
On Wednesday 23 October 2013 01:54:28 Simon Wunderlich wrote:
Instead of handling icmp packets only up to length of icmp_packet_rr, the code should handle any icmp length size. Therefore the length truncating is moved to when the packet is actually sent to userspace (this does not support lengths longer than icmp_packet_rr yet). Longer packets are forwarded without truncating.
This patch also cleans up some parts where the icmp header struct could be used instead of other icmp_packet(_rr) structs to make the code more readable.
Signed-off-by: Simon Wunderlich sw@simonwunderlich.de
Changes to PATCHv3:
- remove unnecesary blank line (buildservers checkpatch complained, mine didn't ...)
Since your patch has been applied already, we will need a patch only fixing this 'regression'. Antonio will merge them together.
Cheers, Marek