CVSROOT:        /cvs
Module name:    src
Changes by:     blamb...@cvs.openbsd.org        2011/04/05 05:48:28

Modified files:
        sys/dev/pci    : hifn7751.c safe.c ubsec.c 
        sys/kern       : uipc_mbuf.c uipc_mbuf2.c 
        sys/net80211   : ieee80211_crypto_ccmp.c ieee80211_crypto_tkip.c 
                         ieee80211_crypto_wep.c ieee80211_input.c 
        sys/netinet6   : icmp6.c ip6_output.c 
        sys/sys        : mbuf.h 

Log message:
Passing M_WAITOK to mbuf functions is supposed to be a contract between
the caller and the function that the function will not fail to allocate
memory and return a NULL pointer. However, m_dup_pkthdr() violates
this contract, making it possible for functions that pass M_WAITOK to
be surprised in ways that hurt.

Fix this by passing the wait flag all the way down the functions that
actually do the allocation for m_dup_pkthdr() so that we won't be
surprised.

man page update forthcoming

ok claudio@

Reply via email to