Pull request #5127: test: fix comparison for pointers in check tests

Merge in SNORT/snort3 from ~ANOROKH/snort3:fix_mac_os to master

Squashed commit of the following:

commit 1d878c766464d72b5ecb4f77581255f1521ba9d5
Author: anorokh <anorokh@cisco.com>
Date:   Tue Jan 27 01:29:23 2026 +0100

    test: fix comparison for pointers in check tests

diff --git a/src/network_inspectors/appid/test/appid_api_test.cc b/src/network_inspectors/appid/test/appid_api_test.cc
index 33690c9b3a6dd551fd0a2137659edb2c11afbc1f..58025131fce86b21dbc499c1f896734c8a4aba33 100644 (file)
--- a/src/network_inspectors/appid/test/appid_api_test.cc
+++ b/src/network_inspectors/appid/test/appid_api_test.cc
@@ -396,10 +396,10 @@ TEST(appid_api, ssl_app_group_id_lookup_zero_len_data)
         test_zero_tls_data, test_zero_tls_data, true, service, client, payload);
 
     CHECK_TRUE(val);
-    CHECK_EQUAL(mock_session->tsession->get_tls_sni(), nullptr);
-    CHECK_EQUAL(mock_session->tsession->get_tls_first_alt_name(), nullptr);
-    CHECK_EQUAL(mock_session->tsession->get_tls_cname(), nullptr);
-    CHECK_EQUAL(mock_session->tsession->get_tls_org_unit(), nullptr);
+    POINTERS_EQUAL(mock_session->tsession->get_tls_sni(), nullptr);
+    POINTERS_EQUAL(mock_session->tsession->get_tls_first_alt_name(), nullptr);
+    POINTERS_EQUAL(mock_session->tsession->get_tls_cname(), nullptr);
+    POINTERS_EQUAL(mock_session->tsession->get_tls_org_unit(), nullptr);
     mock().checkExpectations();
 }
 

diff --git a/src/protocols/test/ssl_protocol_test.cc b/src/protocols/test/ssl_protocol_test.cc
index 7a319ef709c2c9a60da447658c21ece6c707bc14..64f8600662382e6edd9a55490b7bfb4a412db76f 100644 (file)
--- a/src/protocols/test/ssl_protocol_test.cc
+++ b/src/protocols/test/ssl_protocol_test.cc
@@ -107,7 +107,7 @@ TEST(ssl_protocol_tests, cert_data_incomplete_len_2)
     test_data.certs_len = 2;
     auto result = parse_server_certificates(&test_data);
     CHECK_EQUAL(true, result);
-    CHECK_EQUAL(nullptr, test_data.certs_data);
+    POINTERS_EQUAL(nullptr, test_data.certs_data);
     CHECK_EQUAL(0, test_data.certs_len);
 }