ar5523: make buffer size variable unsigned
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 30 Oct 2012 18:04:26 +0000 (21:04 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 30 Oct 2012 20:17:54 +0000 (16:17 -0400)
A negative buffer size doesn't make sense and it breaks this check in
ar5523_get_max_rxsz():

if (!ar->rxbufsz || ar->rxbufsz > AR5523_SANE_RXBUFSZ) { ...

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ar5523/ar5523.h

index 6086ba3..00c6fd3 100644 (file)
@@ -122,7 +122,7 @@ struct ar5523 {
 
        struct work_struct      rx_refill_work;
 
-       int                     rxbufsz;
+       unsigned int            rxbufsz;
        u8                      serial[16];
 
        struct ieee80211_channel channels[14];