[media] drxk: fix warning: ‘status’ may be used uninitialized in this function
authorMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 3 Jul 2011 21:21:59 +0000 (18:21 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 27 Jul 2011 20:55:45 +0000 (17:55 -0300)
One of the problems of the old CHECK_ERROR is that it was hiding
the status parameter. Maybe due to that, on a few places, the return
code might lead to return incorrect status:

drivers/media/dvb/frontends/drxk_hard.c: In function ‘load_microcode.clone.0’:
drivers/media/dvb/frontends/drxk_hard.c:1281: warning: ‘status’ may be used uninitialized in this function
drivers/media/dvb/frontends/drxk_hard.c:1281: note: ‘status’ was declared here
drivers/media/dvb/frontends/drxk_hard.c: In function ‘GetLockStatus’:
drivers/media/dvb/frontends/drxk_hard.c:1792: warning: ‘status’ may be used uninitialized in this function
drivers/media/dvb/frontends/drxk_hard.c: In function ‘Start.clone.7’:
drivers/media/dvb/frontends/drxk_hard.c:1734: warning: ‘status’ may be used uninitialized in this function

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/frontends/drxk_hard.c

index f7ae900811149ba81d1b0858ae9d8ae4b8a4cbff..24f9897077a0c7a51f5126314ffc6bebd63485c3 100644 (file)
@@ -1278,7 +1278,7 @@ static int DownloadMicrocode(struct drxk_state *state,
        u16 BlockCRC;
        u32 offset = 0;
        u32 i;
-       int status;
+       int status = 0;
 
        /* down the drain (we don care about MAGIC_WORD) */
        Drain = (pSrc[0] << 8) | pSrc[1];
@@ -1731,7 +1731,7 @@ static int SetOperationMode(struct drxk_state *state,
 static int Start(struct drxk_state *state, s32 offsetFreq,
                 s32 IntermediateFrequency)
 {
-       int status;
+       int status = 0;
 
        do {
                u16 IFreqkHz;
@@ -1789,7 +1789,7 @@ static int ShutDown(struct drxk_state *state)
 static int GetLockStatus(struct drxk_state *state, u32 *pLockStatus,
                         u32 Time)
 {
-       int status;
+       int status = 0;
 
        if (pLockStatus == NULL)
                return -1;