On Sunday 07 August 2005 06:26, Patrick McHardy wrote:
> > Anyway, do we not want BUG_ON(!atomic_read(&skb->users)) at the beginning
> > of kfree_skb, since we rely on it?
>
> Why do you care if skb->users is 0 or 1 in __kfree_skb()?

Because I am a neatness freak and I like to check things that inattentive 
coders can easily get wrong.  But the question above is not about that, it is 
about checking for possible calls where skb->users is already zero and 
thereby catching the double free early instead of letting it slide further 
into the innards of the machine.

Regards,

Daniel
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to