[PATCH 233/510] Staging: bcm: signedness bug in InitCardAndDownloadFirmware()
Greg Kroah-Hartman
gregkh at suse.de
Mon Jan 10 12:39:17 PST 2011
From: Dan Carpenter <error27 at gmail.com>
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 at gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh at suse.de>
---
drivers/staging/bcm/Misc.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/staging/bcm/Misc.c b/drivers/staging/bcm/Misc.c
index 023c511..9588fbe 100644
--- 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
--
1.7.3.2
More information about the devel
mailing list