diff options
| author | David Benjamin <davidben@chromium.org> | 2015-10-24 01:49:31 +0300 |
|---|---|---|
| committer | Adam Langley <alangley@gmail.com> | 2015-10-26 22:41:58 +0300 |
| commit | dc4a554b2c0dfbba253ba6b7136769ab7a54f2dd (patch) | |
| tree | 9f7c352a5e5444690b38eae520bd870e73ef9317 /crypto/x509 | |
| parent | 79680ffaed9df169d73086bc8a9ce2d81483d428 (diff) | |
Remove dead code in x509_lu.c.
See also upstream's b62a2f8a373d1889672599834acf95161f2883ce, though
upstream left the lock calls in by accident. Otherwise, the change
appears to be correct. I see no side effects of x509_object_idx_cnt
beyond the return value and *pnmatch, both of which are discarded.
Change-Id: Ic2124a733a61591bd1b264164726ce6c69ce10c9
Reviewed-on: https://boringssl-review.googlesource.com/6347
Reviewed-by: Adam Langley <alangley@gmail.com>
Diffstat (limited to 'crypto/x509')
| -rw-r--r-- | crypto/x509/x509_lu.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/crypto/x509/x509_lu.c b/crypto/x509/x509_lu.c index 1bdcddf3..8d4b1634 100644 --- a/crypto/x509/x509_lu.c +++ b/crypto/x509/x509_lu.c @@ -555,13 +555,8 @@ STACK_OF(X509_CRL)* X509_STORE_get1_crls(X509_STORE_CTX *ctx, X509_NAME *nm) sk = sk_X509_CRL_new_null(); if (sk == NULL) return NULL; - CRYPTO_MUTEX_lock_write(&ctx->ctx->objs_lock); - /* Check cache first */ - idx = x509_object_idx_cnt(ctx->ctx->objs, X509_LU_CRL, nm, &cnt); - /* Always do lookup to possibly add new CRLs to cache - */ - CRYPTO_MUTEX_unlock(&ctx->ctx->objs_lock); + /* Always do lookup to possibly add new CRLs to cache. */ if (!X509_STORE_get_by_subject(ctx, X509_LU_CRL, nm, &xobj)) { sk_X509_CRL_free(sk); |