[PATCH] memdisk: Fix "might be used uninitialized" warnings

Shao Miller
Fri, 11 Dec 2009 01:26:59 -0000

The checksum_buf function was providing an incorrect checksum
on at least one build.

Signed-off-by: Shao Miller <shao.miller at yrdsb.edu.on.ca>
---
 memdisk/setup.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/memdisk/setup.c b/memdisk/setup.c
index ea17afd..b8eadfb 100644
--- a/memdisk/setup.c
+++ b/memdisk/setup.c
@@ -592,7 +592,7 @@ static const struct geometry
*get_disk_image_geometry(uint32_t where,
 		while (!ok) {
 		    /* Assume it''s a floppy drive, guess a geometry */
 		    unsigned int type, track;
-		    int c, h, s;
+		    int c, h, s = 0;

 		    if (xsectors < 320 * 2) {
 			c = 40;
@@ -795,7 +795,7 @@ static void relocate_rm_code(uint32_t newbase)
 static uint8_t checksum_buf(const void *buf, int count)
 {
     const uint8_t *p = buf;
-    uint8_t c;
+    uint8_t c = 0;

     while (count--)
 	c += *p++;
--
1.5.3.4


--------------090404070504060403030208--