On Tue, May 14, 2019 at 01:11:21AM +0530, Puranjay Mohan wrote:
> @@ -2856,7 +2856,7 @@ static int ieee80211_wpa_set_encryption(struct 
> ieee80211_device *ieee,
>               goto done;
>       }
>  
> -     if (*crypt == NULL || (*crypt)->ops != ops) {
> +     if (!(*crypt) || (*crypt)->ops != ops) {
             ^      ^
Please don't add these parentheses.

>               struct ieee80211_crypt_data *new_crypt;
>  
>               ieee80211_crypt_delayed_deinit(ieee, crypt);

regards,
dan carpenter

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to