On Tue, Jun 07, 2011 at 04:19:08PM +0200, Patrick McHardy wrote: > >>> With the patch below, I haven't been able to reproduce the problem, but > >>> I don't know if I've inadvertantly broken some other behaviour somewhere > >>> deeper in netlink where this is valid. > > > > This is fine, but I'm wondering whether this can really fix the problem > > you've been seeing. Before the packet is reallocated, the length of > > nlmsglen - NLMSGLEN(0) - sizeof(struct ipq_peer_msg) is compared to > > ipq_peer_msg->data_len, so both values need to be wrong. > > ipq_peer_msg->data_len is a size_t, so it's unsigned. > > > > I think what we should additionally do is verify that data_len < 65535 > > since that's the maximum size of an IP packet. > > We're actually already doing this. This makes it even more strange that > you're seeing this problem. Could you send me your testcase? I don't have a standalone test-case, just a generic fuzzing tool that passes sockets to various syscalls. You can clone it from git://git.codemonkey.org.uk/trinity.git/ (the test-random.sh should explain how to use it) Dave. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
