xfrm: Add error handling when nla_put_u32() returns an error

Error handling is missing when call to nla_put_u32() fails.
Handle the error when the call to nla_put_u32() returns an error.

The error was reported by Coverity Scan.
Report:
CID 1601525: (#1 of 1): Unused value (UNUSED_VALUE)
returned_value: Assigning value from nla_put_u32(skb, XFRMA_SA_PCPU, x->pcpu_num)
to err here, but that stored value is overwritten before it can be used

Fixes: 1ddf9916ac09 ("xfrm: Add support for per cpu xfrm state handling.")
Signed-off-by: Everest K.C. <everestkc@everestkc.com.np>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>

diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c
index b6ce2b3c6b8743ca64182eb8e999b81712d664b7..fab18b85af539c88f722227a8dcc54038355c0ce 100644 (file)
--- a/net/xfrm/xfrm_user.c
+++ b/net/xfrm/xfrm_user.c
@@ -2609,8 +2609,11 @@ static int build_aevent(struct sk_buff *skb, struct xfrm_state *x, const struct
        err = xfrm_if_id_put(skb, x->if_id);
        if (err)
                goto out_cancel;
-       if (x->pcpu_num != UINT_MAX)
+       if (x->pcpu_num != UINT_MAX) {
                err = nla_put_u32(skb, XFRMA_SA_PCPU, x->pcpu_num);
+               if (err)
+                       goto out_cancel;
+       }
 
        if (x->dir) {
                err = nla_put_u8(skb, XFRMA_SA_DIR, x->dir);