On Wed, 2008-10-22 at 04:37 -0700, Luis R. Rodriguez wrote: > On Tue, Oct 21, 2008 at 02:02:00AM -0700, Johannes Berg wrote: > > the current documentation doesn't > > note that you need to free it on errors anyway. > > Yes it does. > > > /* Use this call to set the current regulatory domain. Conflicts with > > * multiple drivers can be ironed out later. Caller must've already > > - * kmalloc'd the rd structure. If this calls fails you should kfree() > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > See? > > > - * the passed rd. Caller must hold cfg80211_drv_mutex */ > > + * kmalloc'd the rd structure. Caller must hold cfg80211_drv_mutex */ > > But this is fine too, so long as we keep the docs in sync. Yeah, evidently I screwed up the commit message. Sorry. I think I'd thought it didn't and then only later noticed. I'll resend. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part