diff options
| author | Colin Ian King <colin.king@canonical.com> | 2016-02-27 12:45:26 +0000 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2016-02-29 14:29:03 +0000 |
| commit | 06aae592425701851e02bb850cb9f4997f0ae163 (patch) | |
| tree | 58f5826e067d3cdcf21daf987367bf7602ca3c77 /crypto | |
| parent | 8e1678988897ebcc29b318ed78af4808202772df (diff) | |
| download | lwn-06aae592425701851e02bb850cb9f4997f0ae163.tar.gz lwn-06aae592425701851e02bb850cb9f4997f0ae163.zip | |
PKCS#7: fix unitialized boolean 'want'
The boolean want is not initialized and hence garbage. The default should
be false (later it is only set to true on tne sinfo->authattrs check).
Found with static analysis using CoverityScan
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/asymmetric_keys/pkcs7_parser.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/asymmetric_keys/pkcs7_parser.c b/crypto/asymmetric_keys/pkcs7_parser.c index 3ef62dac9771..cbbd03fd94f8 100644 --- a/crypto/asymmetric_keys/pkcs7_parser.c +++ b/crypto/asymmetric_keys/pkcs7_parser.c @@ -87,7 +87,7 @@ EXPORT_SYMBOL_GPL(pkcs7_free_message); static int pkcs7_check_authattrs(struct pkcs7_message *msg) { struct pkcs7_signed_info *sinfo; - bool want; + bool want = false; sinfo = msg->signed_infos; if (sinfo->authattrs) { |