Re: [patch] net: renesas: ravb: unintialized return value

2016-12-02 Thread David Miller
From: Dan Carpenter Date: Thu, 1 Dec 2016 23:57:44 +0300 > We want to set the other "err" variable here so that we can return it > later. My version of GCC misses this issue but I caught it with a > static checker. > > Fixes: 9f70eb339f52 ("net: ethernet: renesas:

Re: [patch] net: renesas: ravb: unintialized return value

2016-12-02 Thread Johan Hovold
On Thu, Dec 01, 2016 at 11:57:44PM +0300, Dan Carpenter wrote: > We want to set the other "err" variable here so that we can return it > later. My version of GCC misses this issue but I caught it with a > static checker. > Fixes: 9f70eb339f52 ("net: ethernet: renesas: ravb: fix fixed-link phydev

Re: [patch] net: renesas: ravb: unintialized return value

2016-12-01 Thread Sergei Shtylyov
Hello! On 12/01/2016 11:57 PM, Dan Carpenter wrote: We want to set the other "err" variable here so that we can return it later. My version of GCC misses this issue but I caught it with a static checker. Fixes: 9f70eb339f52 ("net: ethernet: renesas: ravb: fix fixed-link phydev leaks")

[patch] net: renesas: ravb: unintialized return value

2016-12-01 Thread Dan Carpenter
We want to set the other "err" variable here so that we can return it later. My version of GCC misses this issue but I caught it with a static checker. Fixes: 9f70eb339f52 ("net: ethernet: renesas: ravb: fix fixed-link phydev leaks") Signed-off-by: Dan Carpenter ---