From 2b6f63717c3f6495740c343795588e5521ad8586 Mon Sep 17 00:00:00 2001
From: Gregory Cooke <gregorycooke@google.com>
Date: Fri, 16 Feb 2024 14:46:35 +0000
Subject: [PATCH] return bad status on null crl

---
 src/core/lib/security/credentials/tls/grpc_tls_crl_provider.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/core/lib/security/credentials/tls/grpc_tls_crl_provider.cc b/src/core/lib/security/credentials/tls/grpc_tls_crl_provider.cc
index dc004297894..f8adf5f12cc 100644
--- a/src/core/lib/security/credentials/tls/grpc_tls_crl_provider.cc
+++ b/src/core/lib/security/credentials/tls/grpc_tls_crl_provider.cc
@@ -57,7 +57,7 @@ namespace {
 // and add this to it.
 absl::StatusOr<std::string> IssuerFromCrl(X509_CRL* crl) {
   if (crl == nullptr) {
-    return "";
+    return absl::InvalidArgumentError("crl cannot be null");
   }
   X509_NAME* issuer = X509_CRL_get_issuer(crl);
   if (issuer == nullptr) {