Message ID | 1458316291-25705-1-git-send-email-colin.king@canonical.com (mailing list archive) |
---|---|
State | Accepted |
Delegated to: | Herbert Xu |
Headers | show |
On Fri, Mar 18, 2016 at 03:51:31PM +0000, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > The OID_sha224 case is missing a break and it falls through > to the -ENOPKG error default. Since HASH_ALGO_SHA224 seems > to be supported, this looks like an unintentional missing break. > > Fixes: 07f081fb5057 ("PKCS#7: Add OIDs for sha224, sha284 and sha512 hash algos and use them") > Cc: <stable@vger.kernel.org> # 4.2+ > Signed-off-by: Colin Ian King <colin.king@canonical.com> Applied.
Colin King <colin.king@canonical.com> wrote: > From: Colin Ian King <colin.king@canonical.com> > > The OID_sha224 case is missing a break and it falls through > to the -ENOPKG error default. Since HASH_ALGO_SHA224 seems > to be supported, this looks like an unintentional missing break. > > Fixes: 07f081fb5057 ("PKCS#7: Add OIDs for sha224, sha284 and sha512 hash algos and use them") > Cc: <stable@vger.kernel.org> # 4.2+ > Signed-off-by: Colin Ian King <colin.king@canonical.com> Acked-by: David Howells <dhowells@redhat.com> -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/crypto/asymmetric_keys/pkcs7_parser.c b/crypto/asymmetric_keys/pkcs7_parser.c index 40de03f..bdd0d753 100644 --- a/crypto/asymmetric_keys/pkcs7_parser.c +++ b/crypto/asymmetric_keys/pkcs7_parser.c @@ -237,6 +237,7 @@ int pkcs7_sig_note_digest_algo(void *context, size_t hdrlen, break; case OID_sha224: ctx->sinfo->sig.hash_algo = "sha224"; + break; default: printk("Unsupported digest algo: %u\n", ctx->last_oid); return -ENOPKG;