Staging: bcm: signedness bug in InitCardAndDownloadFirmware()

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>

diff --git a/drivers/staging/bcm/Misc.c b/drivers/staging/bcm/Misc.c
index 023c511c00010df26bceedaaaa1a257bbc96d755..9588fbe101d98d5e964496a5b47a1da040e2cddc 100644 (file)
--- a/drivers/staging/bcm/Misc.c
+++ b/drivers/staging/bcm/Misc.c
@@ -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