* [patch] Staging: bcm: signedness bug in
@ 2010-11-13 8:24 Dan Carpenter
0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2010-11-13 8:24 UTC (permalink / raw)
To: kernel-janitors
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>
diff --git a/drivers/staging/bcm/Misc.c b/drivers/staging/bcm/Misc.c
index 1bb6a1d..440b9cd 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
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2010-11-13 8:24 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-13 8:24 [patch] Staging: bcm: signedness bug in Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).