[PATCH] drivers/block/cciss.c (2.6.35.7): Fixed useless compile warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I am sorry if I sent this wrong, but someone told me to send this to this e-mail 
adress.
There isn't practically any problem, but the compiler can't know that, because 
sn is initialized in a conditional expression (else return snprintf(...)...), 
and it throws those ugly warnings:
drivers/block/cciss.c: In function âdev_show_unique_idâ:
drivers/block/cciss.c:564:7: warning: âsn[0]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[1]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[2]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[3]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[4]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[5]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[6]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[7]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[8]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[9]â may be used uninitialized in this 
function
drivers/block/cciss.c:564:7: warning: âsn[10]â may be used uninitialized in this 

function
drivers/block/cciss.c:564:7: warning: âsn[11]â may be used uninitialized in this 

function
drivers/block/cciss.c:564:7: warning: âsn[12]â may be used uninitialized in this 

function
drivers/block/cciss.c:564:7: warning: âsn[13]â may be used uninitialized in this 

function
drivers/block/cciss.c:564:7: warning: âsn[14]â may be used uninitialized in this 

function
drivers/block/cciss.c:564:7: warning: âsn[15]â may be used uninitialized in this 

function
Just have to initialize sn[] with 0. It is practically useless, but the compiler 

wan't throw that warnings anymore.
Patch downer. Thanks.

Signed-off-by: Ionut Gabriel Popescu <poyo_vl@xxxxxxxxx>
---

--- a/drivers/block/cciss.c    2010-09-17 20:17:49.844010958 +0300
+++ b/drivers/block/cciss.c    2010-09-13 19:54:58.000000000 +0300
@@ -561,7 +561,7 @@ static ssize_t dev_show_unique_id(struct
{
     drive_info_struct *drv = to_drv(dev);
     struct ctlr_info *h = to_hba(drv->dev.parent);
-    __u8 sn[16] = {0};
+    __u8 sn[16];
     unsigned long flags;
     int ret = 0;


      
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux