aboutsummaryrefslogtreecommitdiff
path: root/drivers/staging/sep
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2009-12-30 18:10:18 +0200
committerGreg Kroah-Hartman <gregkh@suse.de>2010-03-03 16:42:39 -0800
commit88e05d854cbbd583d528be9295a74586f920492b (patch)
treec650ae893178a341ec3c1b598f1a564bd436ac67 /drivers/staging/sep
parent00367a6d7caae9c6b077e177aeebad5b0a91ea62 (diff)
Staging: rtl8192su: buffer overflow in r8192U_core.c
There was a buffer overflow in the original code. rf_path was 2 and it should have been only 0 or 1. I don't have the hardware for this, so I can't test it. Looking at the code, there are two almost identical sections for updating the hal variables. The first one was clearly wrong and had the array overflow as well. The second one looked correct. I decided to use the second section as is except for whitespace changes. The differences between the two original sections: 1) The second one had more debug output. 2) The second one looped over rf_path instead of corrupting data. 3) The second one had these additional assigments. if (rf_path == 0) { priv->TxPowerLevelOFDM24G[i] = priv->RfTxPwrLevelOfdm1T[rf_path][i] ; priv->TxPowerLevelCCK[i] = priv->RfTxPwrLevelCck[rf_path][i]; } Signed-off-by: Dan Carpenter <error27@gmail.com> Cc: Jerry chuang <wlanfae@realtek.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/sep')
0 files changed, 0 insertions, 0 deletions