Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variable 'rc'

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: Steven Noonan

Subject: Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variable 'rc'

Date: Wednesday, September 10, 2008 - 2:19 pm

On Wed, Sep 10, 2008 at 2:09 PM, Bartlomiej Zolnierkiewicz
<bzolnier@gmail.com> wrote:
quoted text>
> I recall seeing identical patch before:
>
>        http://article.gmane.org/gmane.linux.ide/33107
>
> Does the existing code really results in gcc warning?
>
>

Yes, because GCC thinks it's possible that the for loop has zero
iterations (and hence, 'rc' is never touched).

- Steven
--
unsubscribe noticeTo unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[PATCH] drivers/ide/ide-probe.c: uninitialized variable 'rc', Steven Noonan, (Tue Sep 9, 2:07 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Ingo Molnar, (Tue Sep 9, 11:55 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Bartlomiej Zolnierki ..., (Wed Sep 10, 2:09 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Steven Noonan, (Wed Sep 10, 2:19 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Elias Oltmanns, (Wed Sep 10, 3:40 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Steven Noonan, (Wed Sep 10, 3:44 pm)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Bartlomiej Zolnierki ..., (Sat Sep 27, 8:40 am)

Re: [PATCH] drivers/ide/ide-probe.c: uninitialized variabl ..., Bartlomiej Zolnierki ..., (Sun Oct 5, 9:14 am)


linux-kernel:
Ingo Molnar	Re: [PATCH 0/3] v2 Make hierarchical RCU less IPI-happy and add more tracing
Jeremy Fitzhardinge	Re: Linux 2.6.28.10 and Linux 2.6.29.6 XEN Guest Support Broken x86_64 in BUILD
Nick Piggin	Re: [patch] CFS (Completely Fair Scheduler), v2
Gary Hade	Re: [PATCH 0/5][RFC] Physical PCI slot objects
Dave Johnson	Re: expected behavior of PF_PACKET on NETIF_F_HW_VLAN_RX device?
linux-netdev:
Arnd Bergmann	Re: 64-bit net_device_stats
Stephens, Allan	RE: [PATCH]: tipc: Fix oops on send prior to entering networked mode
frank.blaschka	[patch 3/5] [PATCH] qeth: support z/VM VSWITCH Port Isolation
Wu Fengguang	Re: [PATCH] dm9601: handle corrupt mac address
David Miller	Re: [PATCH net-2.6.24] Fix refcounting problem with netif_rx_reschedule()
git:
Junio C Hamano	Re: [PATCH] [RFC] add Message-ID field to log on git-am operation
Junio C Hamano	Re: Handling large files with GIT
Karl	Re: [ANNOUNCE] pg - A patch porcelain for GIT
Josh Triplett	Re: [RFC][PATCH 00/10] Sparse: Git's "make check" target
Pierre Habouzit	Re: [PATCH] git-daemon: more powerful base-path/user-path settings, using formats.
git-commits-head:
Linux Kernel Mailing List	MIPS: RBTX4939: Fix IOC pin-enable register updating
Linux Kernel Mailing List	regulator: update email address for Liam Girdwood
Linux Kernel Mailing List	[SCSI] ipr: add message to error table
Linux Kernel Mailing List	powerpc/32: Wire up the trampoline code for kdump
Linux Kernel Mailing List	USB: omap_udc: sync with OMAP tree
openbsd-misc:
Josh Grosse	Re: error : pkg add phpMyAdmin
Brian Candler	Re: OBSD's perspective on SELinux
Jacob Meuser	Re: /dev/audio: Device busy
David Vasek	Re: Inexpensive, low power, "wall wart" computer
William Boshuck	Re: Richard Stallman...