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) ||
>

Reply via email to