Re: [PATCH 1/1] net: hns: avoid null pointer dereference

From: David Miller
Date: Thu May 19 2016 - 14:26:19 EST


From: Heinrich Schuchardt <xypron.glpk@xxxxxx>
Date: Tue, 17 May 2016 22:01:15 +0200

> In the statement
> assert(priv || priv->ae_handle);
> the right side of || is only evaluated if priv is null.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@xxxxxx>

I agree with others that this assert() is pretty useless and should
simply be removed.