crypto: user - no parsing of CRYPTO_MSG_GETALG
authorStephan Mueller <smueller@chronox.de>
Mon, 16 May 2016 00:53:36 +0000 (02:53 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Tue, 31 May 2016 08:41:47 +0000 (16:41 +0800)
The CRYPTO_MSG_GETALG netlink message type provides a buffer to the
kernel to retrieve information from the kernel. The data buffer will not
provide any input and will not be read. Hence the nlmsg_parse is not
applicable to this netlink message type.

This patch fixes the following kernel log message when using this
netlink interface:

netlink: 208 bytes leftover after parsing attributes in process `XXX'.

Patch successfully tested with libkcapi from [1] which uses
CRYPTO_MSG_GETALG to obtain cipher-specific information from the kernel.

[1] http://www.chronox.de/libkcapi.html

Signed-off-by: Stephan Mueller <smueller@chronox.de>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
crypto/crypto_user.c

index 43fe85f20d577b4f3d1bbd6576b6d752bc578531..f71960dea882291082d3c47c051c83b9feb04784 100644 (file)
@@ -516,10 +516,12 @@ static int crypto_user_rcv_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
                return err;
        }
 
-       err = nlmsg_parse(nlh, crypto_msg_min[type], attrs, CRYPTOCFGA_MAX,
-                         crypto_policy);
-       if (err < 0)
-               return err;
+       if (type != (CRYPTO_MSG_GETALG - CRYPTO_MSG_BASE)) {
+               err = nlmsg_parse(nlh, crypto_msg_min[type], attrs,
+                                 CRYPTOCFGA_MAX, crypto_policy);
+               if (err < 0)
+                       return err;
+       }
 
        if (link->doit == NULL)
                return -EINVAL;