Merge pull request #158 from redpanda-data/mboquard/core-8474

tls: Do not place functional code in assert
redpanda-data · Dec 6, 2024 · f194d34 · f194d34
2 parents 6a73b6f + 3652787
commit f194d34
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/net/ossl.cc b/src/net/ossl.cc
@@ -346,7 +346,9 @@ class tls::certificate_credentials::impl {
     }()) {
         // The static verify_callback above will use the stored pointer to 'this' to store the last
         // observed x509 certificate
-        assert(X509_STORE_set_ex_data(_creds.get(), credential_store_idx, this) == 1);
+        [[maybe_unused]] auto res =
+            X509_STORE_set_ex_data(_creds.get(), credential_store_idx, this);
+        assert(res == 1);
     }