QUIC APL: Move NULL pointer check to quic_impl.c

Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/23360)

diff --git a/ssl/quic/quic_impl.c b/ssl/quic/quic_impl.c
index 84214bd5f7e33e3c0d8cb33e5aa0ce8a91f2b167..de39a5c7aa750d957ff0af5d48490b601d66a891 100644 (file)
--- a/ssl/quic/quic_impl.c
+++ b/ssl/quic/quic_impl.c
@@ -3300,6 +3300,10 @@ int ossl_quic_get_value_uint(SSL *s, uint32_t class_, uint32_t id,
     if (!expect_quic_conn_only(s, &ctx))
         return 0;
 
+    if (value == NULL)
+        return QUIC_RAISE_NON_NORMAL_ERROR(&ctx,
+                                           ERR_R_PASSED_INVALID_ARGUMENT, NULL);
+
     switch (id) {
     case SSL_VALUE_QUIC_IDLE_TIMEOUT:
         return qc_getset_idle_timeout(&ctx, class_, value, NULL);

diff --git a/ssl/ssl_lib.c b/ssl/ssl_lib.c
index 539d8568f75fb9e251651f568ca115c7a1d64d9d..7d618bd7006a53395316e83528e01c939696c316 100644 (file)
--- a/ssl/ssl_lib.c
+++ b/ssl/ssl_lib.c
@@ -7639,11 +7639,6 @@ int SSL_get_conn_close_info(SSL *s, SSL_CONN_CLOSE_INFO *info,
 int SSL_get_value_uint(SSL *s, uint32_t class_, uint32_t id,
                        uint64_t *value)
 {
-    if (value == NULL) {
-        ERR_raise(ERR_LIB_SSL, ERR_R_PASSED_INVALID_ARGUMENT);
-        return 0;
-    }
-
 #ifndef OPENSSL_NO_QUIC
     if (IS_QUIC(s))
         return ossl_quic_get_value_uint(s, class_, id, value);