Messages in this thread |  | | From | kuznet@ms2 ... | Subject | Re: AX25/socket kernel PATCHes | Date | Sat, 29 Dec 2001 23:37:18 +0300 (MSK) |
| |
Hello!
> So that it checks > < skb2->data || nh.raw > .. > > Let me know if that fixes it. It shouldn't but it would be good to know if > we are picking up header only frames somewhere, or nh.raw has accidentally > been pushed on one header too far.
Indeed... This fix would be right in any case.
I want to remind why this check is made: this pointer must not show to heavens and it must not be NULL. This property was not held in the past, hence this bug trap was inserted. It can be relaxed, provided it preserves its sanitizing function.
Also, note that real value of nh.raw is private to protocol. Particularly, it can be equal to mac.raw. The best thing to do is to set it to the same position as it would have if the same packet appeared on input. But it is not an absolute requirement, as soon as raw packet socket cannot hold it.
Alexey - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |