xfrm: add extack to verify_policy_dir

Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>

diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c
index fa6024b..0042b77 100644 (file)
--- a/net/xfrm/xfrm_user.c
+++ b/net/xfrm/xfrm_user.c
@@ -1481,7 +1481,7 @@ out_noput:
        return err;
 }
 
-static int verify_policy_dir(u8 dir)
+static int verify_policy_dir(u8 dir, struct netlink_ext_ack *extack)
 {
        switch (dir) {
        case XFRM_POLICY_IN:
@@ -1490,6 +1490,7 @@ static int verify_policy_dir(u8 dir)
                break;
 
        default:
+               NL_SET_ERR_MSG(extack, "Invalid policy direction");
                return -EINVAL;
        }
 
@@ -1566,7 +1567,7 @@ static int verify_newpolicy_info(struct xfrm_userpolicy_info *p,
                return -EINVAL;
        }
 
-       ret = verify_policy_dir(p->dir);
+       ret = verify_policy_dir(p->dir, extack);
        if (ret)
                return ret;
        if (p->index && (xfrm_policy_id2dir(p->index) != p->dir)) {
@@ -2102,7 +2103,7 @@ static int xfrm_get_policy(struct sk_buff *skb, struct nlmsghdr *nlh,
        if (err)
                return err;
 
-       err = verify_policy_dir(p->dir);
+       err = verify_policy_dir(p->dir, extack);
        if (err)
                return err;
 
@@ -2407,7 +2408,7 @@ static int xfrm_add_pol_expire(struct sk_buff *skb, struct nlmsghdr *nlh,
        if (err)
                return err;
 
-       err = verify_policy_dir(p->dir);
+       err = verify_policy_dir(p->dir, extack);
        if (err)
                return err;