Re: [PATCH] forcedeth: fix random memory scribbling bug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Linus Torvalds wrote:
However, that

	"skb->end - skb->data"

calculation is a bit strange. It correctly maps the whole skb, but

nod


wouldn't it make more sense to use the length we actually tell the card to use? In other words, wouldn't it be a whole lot more sensible and logical to use

	np->rx_buf_sz

instead? That's the value we use for allocation and that's the size we tell the card we have.

That's the sort of thing I prefer.


Of course, on the alloc path, it seems to add an additional "NV_RX_ALLOC_PAD" thing, so maybe the "end-data" thing makes sense.

None of the other ethernet drivers do 'end - data', which is why I hesitated quite a bit on this change.

	Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux