Re: [PATCH -tip] drivers/serial/8250.c: 'i' may be useduninitialized

From: Ingo Molnar
Date: Wed Oct 01 2008 - 04:02:08 EST



( Alan Cc:-ed. Seems like gcc bogosity - so your solution of using
uninitialized_var() is the correct way to annotate this. )

* Steven Noonan <steven@xxxxxxxxxxxxxx> wrote:

> Signed-off-by: Steven Noonan <steven@xxxxxxxxxxxxxx>
> ---
> drivers/serial/8250.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c
> index 356c2a2..b51e91e 100644
> --- a/drivers/serial/8250.c
> +++ b/drivers/serial/8250.c
> @@ -1599,7 +1599,7 @@ static int serial_link_irq_chain(struct uart_8250_port *up)
>
> static void serial_unlink_irq_chain(struct uart_8250_port *up)
> {
> - struct irq_info *i;
> + struct irq_info *uninitialized_var(i);
> struct hlist_node *n;
> struct hlist_head *h;
>
> --
> 1.6.0.2
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/