On Tue, Nov 24, 2015 at 01:43:38AM +0200, Andy Shevchenko wrote:
> On Mon, Nov 23, 2015 at 2:32 PM, Sudip Mukherjee
> wrote:
> > We were dereferencing cmd first and checking for NULL later. Lets first
> > check for NULL.
>
> However the patch is valid due to reducing error prone part, the
> commi
On Mon, Nov 23, 2015 at 2:32 PM, Sudip Mukherjee
wrote:
> We were dereferencing cmd first and checking for NULL later. Lets first
> check for NULL.
However the patch is valid due to reducing error prone part, the
commit message seems wrong.
There is no dereferencing, it is a simple pointer arithm
We were dereferencing cmd first and checking for NULL later. Lets first
check for NULL.
Signed-off-by: Sudip Mukherjee
---
drivers/net/wireless/marvell/libertas/cfg.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/marvell/libertas/cfg.c
b/drivers/net/