Mon, Apr 10, 2017 at 03:11:06PM CEST, weiyj...@gmail.com wrote: >From: Wei Yongjun <weiyongj...@huawei.com> > >Fix the retrn value check which testing the wrong variable
^^^^^ typo Otherwise, Acked-by: Jiri Pirko <j...@mellanox.com> >in devlink_dpipe_header_put(). > >Fixes: 1555d204e743 ("devlink: Support for pipeline debug (dpipe)") >Signed-off-by: Wei Yongjun <weiyongj...@huawei.com> >--- > net/core/devlink.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/net/core/devlink.c b/net/core/devlink.c >index 24b7660..0afac58 100644 >--- a/net/core/devlink.c >+++ b/net/core/devlink.c >@@ -2031,7 +2031,7 @@ static int devlink_dpipe_header_put(struct sk_buff *skb, > int err; > > header_attr = nla_nest_start(skb, DEVLINK_ATTR_DPIPE_HEADER); >- if (!header) >+ if (!header_attr) > return -EMSGSIZE; > > if (nla_put_string(skb, DEVLINK_ATTR_DPIPE_HEADER_NAME, header->name) || >