Ocfs2 devel - Sep 2010 - [PATCH] /fs/ocfs2/slot_map.c (2.6.35.6): Uninitialized variable

When I tried to compile I got the following warning:
fs/ocfs2/slot_map.c: In function ?ocfs2_init_slot_info?:
fs/ocfs2/slot_map.c:360: warning: ?bytes? may be used uninitialized in this 
function
fs/ocfs2/slot_map.c:360: note: ?bytes? was declared here 
Compiler: gcc version 4.4.3 (GCC) on Mandriva
I'm not sure why this warning occurs, I think compiler don't know that
variable
"bytes" is initialized when it is sent by reference to 
ocfs2_slot_map_physical_size and it throws that ugly warning.
However, a simple initialization of "bytes" variable with 0 will fix
it.
Patch is downer, I also attached it because there are problems with Yahoo! Mail.

Signed-off-by: Ionut Gabriel Popescu <poyo_vl at yahoo.com>
---

--- a/fs/ocfs2/slot_map.c    2010-09-27 22:27:07.605568537 +0300
+++ b/fs/ocfs2/slot_map.c    2010-09-27 22:27:29.342567489 +0300
@@ -357,7 +357,7 @@ static int ocfs2_map_slot_buffers(struct
 {
     int status = 0;
     u64 blkno;
-    unsigned long long blocks, bytes;
+    unsigned long long blocks, bytes = 0;
     unsigned int i;
     struct buffer_head *bh;


      
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.diff
Type: application/octet-stream
Size: 341 bytes
Desc: not available
Url :
http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20100927/b34b17b6/attachment.obj

On Mon, Sep 27, 2010 at 01:28:37PM -0700, Poyo VL wrote:
> When I tried to compile I got the following warning:
> fs/ocfs2/slot_map.c: In function ?ocfs2_init_slot_info?:
> fs/ocfs2/slot_map.c:360: warning: ?bytes? may be used uninitialized in this
> function
> fs/ocfs2/slot_map.c:360: note: ?bytes? was declared here 
> Compiler: gcc version 4.4.3 (GCC) on Mandriva
> I'm not sure why this warning occurs, I think compiler don't know
that variable
> "bytes" is initialized when it is sent by reference to 
> ocfs2_slot_map_physical_size and it throws that ugly warning.
> However, a simple initialization of "bytes" variable with 0 will
fix it.
> Patch is downer, I also attached it because there are problems with Yahoo!
Mail.
> 
> Signed-off-by: Ionut Gabriel Popescu <poyo_vl at yahoo.com>
This looks reasonable, though using uninitialized_var() might be the
preferred method.

Acked-by: Mark Fasheh <mfasheh at suse.com>


--
Mark Fasheh

On Mon, Sep 27, 2010 at 01:28:37PM -0700, Poyo VL wrote:
> When I tried to compile I got the following warning:
> fs/ocfs2/slot_map.c: In function ?ocfs2_init_slot_info?:
> fs/ocfs2/slot_map.c:360: warning: ?bytes? may be used uninitialized in this
> function
> fs/ocfs2/slot_map.c:360: note: ?bytes? was declared here 
> Compiler: gcc version 4.4.3 (GCC) on Mandriva
> I'm not sure why this warning occurs, I think compiler don't know
that variable
> "bytes" is initialized when it is sent by reference to 
> ocfs2_slot_map_physical_size and it throws that ugly warning.
> However, a simple initialization of "bytes" variable with 0 will
fix it.
> Patch is downer, I also attached it because there are problems with Yahoo!
Mail.
> 
> Signed-off-by: Ionut Gabriel Popescu <poyo_vl at yahoo.com>
	This patch is now in the merge-window branch of ocfs2.git.  In
the future, please send patches inline without MIME.

Joel
-- 

"The lawgiver, of all beings, most owes the law allegiance.  He of all
 men should behave as though the law compelled him.  But it is the
 universal weakness of mankind that what we are given to administer we
 presently imagine we own."
        - H.G. Wells

Joel Becker
Consulting Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127