summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2011-09-19Staging: bcm: Add size maximum size restrictions for IOCTL_IDLE_REQKevin McKinney
In the first alteration, the MAX_CNTL_PKT_SIZE is the maximum size of the control packet in ->Adapter->txctlpacket[] which is defined in InitAdapter(). This caps the size of kmalloc memory allocation. In the second change, this max cap fixes a potential memory corruption bug when subsequent memset and memcpy calls are invoked. Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-19Staging: bcm: Add size minimum size restrictions for IOCTL_IDLE_REQKevin McKinney
If IoBuffer.InputLength is zero then this will cause an Oops when we dereference the ZERO_SIZE_PTR. Or if it's smaller than sizeof(struct link_request) then we would get memory corruption when we set ->PLength in CopyBufferToControlPacket(). Signed-off-by: Kevin McKinney <klmckinney1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-19staging: usbip: fix up api changes that broke windows clientsAlexander Thomas
Revert changes in definitions that were submitted on May 11 2011 and committed on June 7 2011. No reason for the change in these values was given in the patch comment, it broke compatibility with older versions, and was difficult to detect by the simultaneous move of the lines of code. Signed-off-by: Alexander Thomas <alexander.thomas@esaturnus.com> Cc: stable <stable@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: sparse endianness warnings on fullmac bss infoRoland Vossen
Structure that is received from dongle has been annotated. 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>
2011-09-16staging: brcm80211: remove static global variables from dhd_sdio.cArend van Spriel
Several global variables that were set but never initialized have been removed. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove global variable from bcmsdh_sdmmc.cArend van Spriel
The source file contained some static global variables that were needed per device handled by the driver. They have moved to the struct brcmf_sdio_dev for that reason. Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove for_each_bss() macro from brcmfmac driverArend van Spriel
The macro was only used once in the code and has been removed for clarity reading the code. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove brcmfmac interface name overrideArend van Spriel
The brcmfmac had a module parameter which could be used to override the name of the wireless net device name and it defaults to 'wlan'. This is considered rather useless, because what is in a name. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove fullmac module_param for ARP offload featureFranky Lin
Use macro to replace global variable for ARP offload configuration Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove _brcmf_sysioc_thread in fullmacFranky Lin
_brcmf_sysioc_thread is handling deferred jobs from 4 callers. Use work queue to handle set_mac_address/set_multicast_list requests. brcmf_add_if/brcmf_del_if never run in atomic context and can call brcmf_op_if directly. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: fixed sparse endianness warnings on fullmac ratespecRoland Vossen
Structure that is sent to the dongle has been annotated. 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>
2011-09-16staging: brcm80211: fixed sparse endianness warnings on fullmac scb parsRoland Vossen
Structure that is sent to the dongle has been annotated. 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>
2011-09-16staging: brcm80211: fixed sparse endianness warnings on fullmac assoc parsRoland Vossen
Structures that are sent to the dongle have been annotated. 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>
2011-09-16staging: brcm80211: fixed sparse endianness warnings on fullmac scan parsRoland Vossen
Three structures that are sent to the dongle have been annotated for correct endianness: struct brcmf_iscan_params, brcmf_scan_params, brcmf_cfg80211_scan_req. 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>
2011-09-16staging: brcm80211: removed set_fs/get_fs/get_ds calls from brcmf_dev_ioctl()Roland Vossen
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>
2011-09-16staging: brcm80211: simplified brcmf_proto_ioctl()Roland Vossen
Removed a function parameter since it was redundant. 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>
2011-09-16staging: brcm80211: remove global variable brcmf_radio_up from fullmacFranky Lin
Remove unnecessary global variable Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove global variable brcmf_roam from fullmacFranky Lin
Remove unnecessary global variable Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove fullmac module_param for packet filter featureFranky Lin
Use constant values for packet filter configuration Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove fullmac module_param brcmf_deferred_txFranky Lin
Remove brcmf_deferred_tx as it should be always 1 Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove fullmac module_param sd_f2_blocksizeFranky Lin
Replaced the global variable with macro. Reported-by: Johannes Berg <johannes@sipsolutions.net> Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16staging: brcm80211: remove SOFTAP code from fullmacFranky Lin
Dead code removal. Reviewed-by: Roland Vossen <rvossen@broadcom.com> Reviewed-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: Franky Lin <frankyl@broadcom.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16drivers/staging/mei/interface.c: take size of pointed value, not pointerJulia Lawall
Sizeof a pointer-typed expression returns the size of the pointer, not that of the pointed data. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression *e; type T; identifier f; @@ f(...,(T)e,..., -sizeof(e) +sizeof(*e) ,...) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: cx25821-video.h: Fix long linesLeonid V. Fedorenchik
Fix long function prototypes and line up arguments nicely. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: cx25821-video.h: Fix indentLeonid V. Fedorenchik
Fix indent of function arguments. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: cx25821-video.h: Line up constantLeonid V. Fedorenchik
Line up integer constant with others. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Replace :? by if-else in cx25821-video.cLeonid V. Fedorenchik
Replace :? operator by equivalent if-else statement where in improves readability. Don't add else branch if it is not needed. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Fix braces around statements in cx25821-video.cLeonid V. Fedorenchik
Delete braces around single statements. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 35Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 34Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 33Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 32Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 31Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 30Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 29Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 28Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 27Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 26Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 25Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 24Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 23Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 22Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 21Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 20Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 19Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 18Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 17Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 16Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 15Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
2011-09-16Staging: cx25821: Change indent with spaces to tabs 14Leonid V. Fedorenchik
Change indent with spaces to tabs. Signed-off-by: Leonid V. Fedorenchik <leonidsbox@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>