Re: [PATCH] mpc832x_rdb.c: prevent using uninitialized variable

2007-07-27 Thread Anton Vorontsov
On Thu, Jul 26, 2007 at 11:48:33AM -0500, Scott Wood wrote: > On Thu, Jul 26, 2007 at 06:01:39PM +0400, Anton Vorontsov wrote: > > If CONFIG_PCI undefined, np will be used uninitialized, thereby > > find_node_by_name(np, "par_io") will fail. > > This is the wrong fix -- you should be passing NULL

Re: [PATCH] mpc832x_rdb.c: prevent using uninitialized variable

2007-07-26 Thread Scott Wood
On Thu, Jul 26, 2007 at 06:01:39PM +0400, Anton Vorontsov wrote: > If CONFIG_PCI undefined, np will be used uninitialized, thereby > find_node_by_name(np, "par_io") will fail. This is the wrong fix -- you should be passing NULL to the par_io call rather than np. Otherwise, if the par_io is earlie