From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Sun, 07 Mar 2010 15:27:57 -0800 (PST) Message-Id: <20100307.152757.142897943.davem@davemloft.net> From: David Miller In-Reply-To: <20100306122031.GA15864@gondor.apana.org.au> References: <20100306111408.GJ4958@bicker> <20100306122031.GA15864@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Subject: Re: [Bridge] [patch] bridge: cleanup: remove unneed check List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: herbert@gondor.apana.org.au Cc: netdev@vger.kernel.org, kernel-janitors@vger.kernel.org, bridge@lists.linux-foundation.org, error27@gmail.com From: Herbert Xu Date: Sat, 6 Mar 2010 20:20:32 +0800 > On Sat, Mar 06, 2010 at 02:14:09PM +0300, Dan Carpenter wrote: >> We dereference "port" on the lines immediately before and immediately >> after the test so port should hopefully never be null here. >> >> Signed-off-by: Dan Carpenter > > Signed-off-by: Herbert Xu > > Yes this looks correct to me. Applied.