Re: [patch 6/7] drivers/media/dvb/frontends/stv090x.c: fix use-uninitialised

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

 



On Wed, Feb 3, 2010 at 2:40 AM,  <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>
> drivers/media/dvb/frontends/stv090x.c: In function 'stv090x_blind_search':
> drivers/media/dvb/frontends/stv090x.c:1967: warning: 'coarse_fail' may be used uninitialized in this function
>
> Cc: Manu Abraham <manu@xxxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
>
>  drivers/media/dvb/frontends/stv090x.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff -puN drivers/media/dvb/frontends/stv090x.c~drivers-media-dvb-frontends-stv090xc-fix-use-uninitialised drivers/media/dvb/frontends/stv090x.c
> --- a/drivers/media/dvb/frontends/stv090x.c~drivers-media-dvb-frontends-stv090xc-fix-use-uninitialised
> +++ a/drivers/media/dvb/frontends/stv090x.c
> @@ -1964,7 +1964,8 @@ static int stv090x_blind_search(struct s
>        u32 agc2, reg, srate_coarse;
>        s32 cpt_fail, agc2_ovflw, i;
>        u8 k_ref, k_max, k_min;
> -       int coarse_fail, lock;
> +       int coarse_fail = 0;
> +       int lock;
>
>        k_max = 110;
>        k_min = 10;
> _
>

Looks good

Reviewed-by: Manu Abraham <manu@xxxxxxxxxxx>
Acked-by: Manu Abraham <manu@xxxxxxxxxxx>

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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux