lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 24 Apr 2008 09:16:31 +0300 From: Jussi Kivilinna <jussi.kivilinna@...et.fi> To: Roel Kluin <12o3l@...cali.nl> Cc: linux-wireless@...r.kernel.org, lkml <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] wireless: rndis_wlan: modparam_workaround_interval is never below 0. Quoting Roel Kluin <12o3l@...cali.nl>: > priv->param_workaround_interval is unsigned, > modparam_workaround_interval not. > the former is never < 0. > > Signed-off-by: Roel Kluin <12o3l@...cali.nl> Thanks. Acked-by: Jussi Kivilinna <jussi.kivilinna@...et.fi> > --- > diff --git a/drivers/net/wireless/rndis_wlan.c > b/drivers/net/wireless/rndis_wlan.c > index 977751f..d0b1fb1 100644 > --- a/drivers/net/wireless/rndis_wlan.c > +++ b/drivers/net/wireless/rndis_wlan.c > @@ -2402,7 +2402,6 @@ static int bcm4320_early_init(struct usbnet *dev) > priv->param_power_output = modparam_power_output; > priv->param_roamtrigger = modparam_roamtrigger; > priv->param_roamdelta = modparam_roamdelta; > - priv->param_workaround_interval = modparam_workaround_interval; > > priv->param_country[0] = toupper(priv->param_country[0]); > priv->param_country[1] = toupper(priv->param_country[1]); > @@ -2425,8 +2424,10 @@ static int bcm4320_early_init(struct usbnet *dev) > else if (priv->param_roamdelta > 2) > priv->param_roamdelta = 2; > > - if (priv->param_workaround_interval < 0) > + if (modparam_workaround_interval < 0) > priv->param_workaround_interval = 500; > + else > + priv->param_workaround_interval = modparam_workaround_interval; > > rndis_set_config_parameter_str(dev, "Country", priv->param_country); > rndis_set_config_parameter_str(dev, "FrameBursting", > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists