Staging: bcm: signedness bug in InitCardAndDownloadFirmware()
authorDan Carpenter <error27@gmail.com>
Sat, 13 Nov 2010 08:24:22 +0000 (11:24 +0300)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 16 Nov 2010 20:31:50 +0000 (12:31 -0800)
status is used to store negative error codes throughout.  The only place
where this is a runtime bug is if create_worker_threads() fails.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/bcm/Misc.c

index 023c511..9588fbe 100644 (file)
@@ -1079,7 +1079,7 @@ int run_card_proc(PMINI_ADAPTER ps_adapter )
 int InitCardAndDownloadFirmware(PMINI_ADAPTER ps_adapter)
 {
 
-       UINT status = STATUS_SUCCESS;
+       int status;
        UINT value = 0;
        /*
         * Create the threads first and then download the