Skip to content

Commit

Permalink
staging: brcm80211: removed set_fs/get_fs/get_ds calls from brcmf_dev…
Browse files Browse the repository at this point in the history
…_ioctl()

brcmf_dev_ioctl() does not provide a user space buffer to
brcmf_netdev_ioctl_priv. Hence these calls are not necessary.

Reported-by: Johannes Berg <johannes@sipsolutions.net>
Reviewed-by: Arend van Spriel <arend@broadcom.com>
Reviewed-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Franky Lin <frankyl@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Roland Vossen authored and Greg Kroah-Hartman committed Sep 16, 2011
1 parent 862c307 commit 90b3b11
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions drivers/staging/brcm80211/brcmfmac/wl_cfg80211.c
Original file line number Diff line number Diff line change
Expand Up @@ -251,18 +251,14 @@ static s32
brcmf_dev_ioctl(struct net_device *dev, u32 cmd, void *arg, u32 len)
{
struct brcmf_ioctl ioc;
mm_segment_t fs;
s32 err = 0;

memset(&ioc, 0, sizeof(ioc));
ioc.cmd = cmd;
ioc.buf = arg;
ioc.len = len;

fs = get_fs();
set_fs(get_ds());
err = brcmf_netdev_ioctl_priv(dev, &ioc);
set_fs(fs);

return err;
}
Expand Down

0 comments on commit 90b3b11

Please sign in to comment.