diff options
| author | Kees Cook <keescook@chromium.org> | 2013-07-03 15:01:15 -0700 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2013-07-27 05:34:28 +0100 |
| commit | 00bbe5df5c7cc4ddbd0e735b9056eec8f4b2ce0f (patch) | |
| tree | 3100d66291ef10f7fa94dcbc7bfd877ec1c71a15 /crypto | |
| parent | b442223040adf969fd02124c29c856a06cf5649c (diff) | |
| download | kernel-common-00bbe5df5c7cc4ddbd0e735b9056eec8f4b2ce0f.tar.gz kernel-common-00bbe5df5c7cc4ddbd0e735b9056eec8f4b2ce0f.tar.bz2 kernel-common-00bbe5df5c7cc4ddbd0e735b9056eec8f4b2ce0f.zip | |
crypto: sanitize argument for format string
commit 1c8fca1d92e14859159a82b8a380d220139b7344 upstream.
The template lookup interface does not provide a way to use format
strings, so make sure that the interface cannot be abused accidentally.
Signed-off-by: Kees Cook <keescook@chromium.org>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/algapi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/crypto/algapi.c b/crypto/algapi.c index 54dd4e33b5d6..dc9991f5ab5e 100644 --- a/crypto/algapi.c +++ b/crypto/algapi.c @@ -477,7 +477,8 @@ static struct crypto_template *__crypto_lookup_template(const char *name) struct crypto_template *crypto_lookup_template(const char *name) { - return try_then_request_module(__crypto_lookup_template(name), name); + return try_then_request_module(__crypto_lookup_template(name), "%s", + name); } EXPORT_SYMBOL_GPL(crypto_lookup_template); |