Subject: CVS commit: src/sys
To: None <source-changes@NetBSD.org>
From: David Young <dyoung@netbsd.org>
List: source-changes
Date: 07/06/2005 23:44:17
Module Name: src
Committed By: dyoung
Date: Wed Jul 6 23:44:17 UTC 2005
Modified Files:
src/sys/dev/ic: ral.c rtw.c wi.c
src/sys/dev/pci: if_ipw.c
src/sys/dev/usb: if_ural.c
src/sys/net80211: ieee80211_crypto.c
Log Message:
Avoid an unnecessary API difference between NetBSD and FreeBSD:
back out my change to ieee80211_crypto_encap that made it free its
mbuf argument on error. I had thought it was a bug. It was not.
It's the drivers that are broken. Make an(4), atw(4), ipw(4),
iwi(4), ral(4), rtw(4), ural(4), and wi(4) free the mbuf when
ieee80211_crypto_encap returns NULL. Also, return ath(4) to the
way it was---i.e., free the mbuf.
Thanks to Sam Leffler to pointing out my mistake.
To generate a diff of this commit:
cvs rdiff -r1.2 -r1.3 src/sys/dev/ic/ral.c
cvs rdiff -r1.50 -r1.51 src/sys/dev/ic/rtw.c
cvs rdiff -r1.202 -r1.203 src/sys/dev/ic/wi.c
cvs rdiff -r1.12 -r1.13 src/sys/dev/pci/if_ipw.c
cvs rdiff -r1.2 -r1.3 src/sys/dev/usb/if_ural.c
cvs rdiff -r1.7 -r1.8 src/sys/net80211/ieee80211_crypto.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.