On Wed, Sep 16, 2015 at 07:16:22AM +0000, Kevin Lo wrote:
> Author: kevlo
> Date: Wed Sep 16 07:16:21 2015
> New Revision: 287852
> URL: https://svnweb.freebsd.org/changeset/base/287852
> 
> Log:
>   Remove checks for a NULL return value from M_WAITOK allocations.
> 
> Modified:
>   head/sys/dev/usb/wlan/if_ural.c
> 
> Modified: head/sys/dev/usb/wlan/if_ural.c
> ==============================================================================
> --- head/sys/dev/usb/wlan/if_ural.c   Wed Sep 16 06:23:15 2015        
> (r287851)
> +++ head/sys/dev/usb/wlan/if_ural.c   Wed Sep 16 07:16:21 2015        
> (r287852)
> @@ -566,10 +566,7 @@ ural_vap_create(struct ieee80211com *ic,
>  
>       if (!TAILQ_EMPTY(&ic->ic_vaps))         /* only one at a time */
>               return NULL;
> -     uvp = (struct ural_vap *) malloc(sizeof(struct ural_vap),
> -         M_80211_VAP, M_NOWAIT | M_ZERO);
> -     if (uvp == NULL)
> -             return NULL;
> +     uvp = malloc(sizeof(struct ural_vap), M_80211_VAP, M_NOWAIT | M_ZERO);
>       vap = &uvp->vap;
>       /* enable s/w bmiss handling for sta mode */
>  
> 

This looks like M_NOWAIT prior to and after the commit.

I have no idea if the context here allows sleeping though.

-- 
Mateusz Guzik <mjguzik gmail.com>
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"

Reply via email to