diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-06-08 15:47:49 +0200 |
|---|---|---|
| committer | James Morris <james.l.morris@oracle.com> | 2017-06-09 05:29:45 +0200 |
| commit | 4e880168e9ffb1cdbdb72b3b48ab0324b30c2d62 (patch) | |
| tree | 5a709757c0c2bb76f748d8c734321aa0f71cb69c | |
| parent | security: use READ_ONCE instead of deprecated ACCESS_ONCE (diff) | |
| download | linux-4e880168e9ffb1cdbdb72b3b48ab0324b30c2d62.tar.xz linux-4e880168e9ffb1cdbdb72b3b48ab0324b30c2d62.zip | |
X.509: Fix error code in x509_cert_parse()
We forgot to set the error code on this path so it could result in
returning NULL which leads to a NULL dereference.
Fixes: db6c43bd2132 ("crypto: KEYS: convert public key and digsig asym to the akcipher api")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: James Morris <james.l.morris@oracle.com>
| -rw-r--r-- | crypto/asymmetric_keys/x509_cert_parser.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/crypto/asymmetric_keys/x509_cert_parser.c b/crypto/asymmetric_keys/x509_cert_parser.c index c80765b211cf..dd03fead1ca3 100644 --- a/crypto/asymmetric_keys/x509_cert_parser.c +++ b/crypto/asymmetric_keys/x509_cert_parser.c @@ -102,6 +102,7 @@ struct x509_certificate *x509_cert_parse(const void *data, size_t datalen) } } + ret = -ENOMEM; cert->pub->key = kmemdup(ctx->key, ctx->key_size, GFP_KERNEL); if (!cert->pub->key) goto error_decode; |