diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-04-04 10:31:24 +0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-04-06 01:17:00 +0400 |
commit | 69c1440e864287914c94c244b3d1ca7802089e66 (patch) | |
tree | c23874db446fcb96940f1c3e13b1f73afbd6ccd7 /drivers | |
parent | 2c71c4ff57a6564097bedd45bc589f73da798a3a (diff) | |
download | linux-69c1440e864287914c94c244b3d1ca7802089e66.tar.xz |
staging: csr: info leak in unifi_cfg_get_info()
"cfg_ap_config" has a number of fields which are not cleared before we
copy them to the user. I've added a memset() at the beginning to set
everything to zero.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/csr/unifi_sme.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/csr/unifi_sme.c b/drivers/staging/csr/unifi_sme.c index 7d19e632a5e4..90295035621f 100644 --- a/drivers/staging/csr/unifi_sme.c +++ b/drivers/staging/csr/unifi_sme.c @@ -876,6 +876,8 @@ int unifi_cfg_get_info(unifi_priv_t *priv, unsigned char *arg) { #ifdef CSR_SUPPORT_WEXT_AP uf_cfg_ap_config_t cfg_ap_config; + + memset(&cfg_ap_config, 0, sizeof(cfg_ap_config)); cfg_ap_config.channel = priv->ap_config.channel; cfg_ap_config.beaconInterval = priv->ap_mac_config.beaconInterval; cfg_ap_config.wmmEnabled = priv->ap_mac_config.wmmEnabled; |