]> git.ipfire.org Git - thirdparty/openssh-portable.git/commitdiff
upstream: Null out keys after freeing in tests in the case where we
authordtucker@openbsd.org <dtucker@openbsd.org>
Mon, 29 Sep 2025 07:40:55 +0000 (07:40 +0000)
committerDarren Tucker <dtucker@dtucker.net>
Mon, 29 Sep 2025 21:41:55 +0000 (07:41 +1000)
potentially reuse the variable.  Fixes Coverity CID 405057.

OpenBSD-Regress-ID: c52e86502b33bfa6e448448a74a0217dd519dd58

regress/unittests/sshkey/test_sshkey.c

index 5d5a6bec38433bf7365a714abe10f87171b9876a..bfaa3f66d592856e3580bb4c486c0c57e1752882 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: test_sshkey.c,v 1.30 2025/05/24 06:50:28 dtucker Exp $ */
+/*     $OpenBSD: test_sshkey.c,v 1.31 2025/09/29 07:40:55 dtucker Exp $ */
 /*
  * Regress test for sshkey.h key management API
  *
@@ -258,6 +258,7 @@ sshkey_tests(void)
        k1 = sshkey_new(KEY_UNSPEC);
        ASSERT_PTR_NE(k1, NULL);
        sshkey_free(k1);
+       k1 = NULL;
        TEST_DONE();
 
 #ifdef WITH_OPENSSL
@@ -266,6 +267,7 @@ sshkey_tests(void)
        ASSERT_PTR_NE(k1, NULL);
        ASSERT_PTR_NE(k1->pkey, NULL);
        sshkey_free(k1);
+       k1 = NULL;
        TEST_DONE();
 
 
@@ -275,6 +277,7 @@ sshkey_tests(void)
        ASSERT_PTR_NE(k1, NULL);
        ASSERT_PTR_EQ(k1->pkey, NULL);  /* Can't allocate without NID */
        sshkey_free(k1);
+       k1 = NULL;
        TEST_DONE();
 #endif
 
@@ -285,6 +288,7 @@ sshkey_tests(void)
        ASSERT_PTR_EQ(k1->ed25519_sk, NULL);
        ASSERT_PTR_EQ(k1->ed25519_pk, NULL);
        sshkey_free(k1);
+       k1 = NULL;
        TEST_DONE();
 
        TEST_START("generate KEY_RSA too small modulus");
@@ -306,6 +310,7 @@ sshkey_tests(void)
            SSH_ERR_KEY_LENGTH);
        ASSERT_PTR_EQ(k1, NULL);
        sshkey_free(k1);
+       k1 = NULL;
        TEST_DONE();
 #endif