]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
test_crypto.c: Fix getcwd(…) build error.
authorSean Bright <sean@seanbright.com>
Mon, 27 Feb 2023 21:35:57 +0000 (16:35 -0500)
committerGeorge Joseph <gjoseph@digium.com>
Tue, 28 Feb 2023 13:52:35 +0000 (07:52 -0600)
`getcwd(…)` is decorated with the `warn_unused_result` attribute and
therefore needs its return value checked.

Change-Id: Idcccb20a0abf293202c28633d0e9ee0f6a9dbe93

tests/test_crypto.c

index 848a562b3676050b5a86ef8f2073a49fba995585..8b52c9df383e30b444e68719bf7c06d459efe46d 100644 (file)
@@ -110,10 +110,16 @@ AST_TEST_DEFINE(crypto_rsa_encrypt)
 
        if (!ast_check_command_in_path(command)) {
                ast_test_status_update(test, "couldn't find %s\n", command);
+               ast_test_capture_free(&cap);
+               return res;
+       }
+
+       if (getcwd(wd, sizeof(wd)) == NULL) {
+               ast_test_status_update(test, "Could not determine current working directory\n");
+               ast_test_capture_free(&cap);
                return res;
        }
 
-       getcwd(wd, sizeof(wd));
        snprintf(key_dir, sizeof(key_dir), "%s/%s", wd, "tests/keys");
        push_key_dir((const char *)key_dir);
        snprintf(priv, sizeof(priv), "%s/%s.key", key_dir, keypair1);
@@ -200,10 +206,16 @@ AST_TEST_DEFINE(crypto_rsa_decrypt)
 
        if (!ast_check_command_in_path(command)) {
                ast_test_status_update(test, "couldn't find %s\n", command);
+               ast_test_capture_free(&cap);
+               return res;
+       }
+
+       if (getcwd(wd, sizeof(wd)) == NULL) {
+               ast_test_status_update(test, "Could not determine current working directory\n");
+               ast_test_capture_free(&cap);
                return res;
        }
 
-       getcwd(wd, sizeof(wd));
        snprintf(key_dir, sizeof(key_dir), "%s/%s", wd, "tests/keys");
        push_key_dir((const char *)key_dir);
        snprintf(pub, sizeof(pub), "%s/%s.pub", key_dir, keypair1);
@@ -297,10 +309,16 @@ AST_TEST_DEFINE(crypto_sign)
 
        if (!ast_check_command_in_path(command)) {
                ast_test_status_update(test, "couldn't find %s\n", command);
+               ast_test_capture_free(&cap);
+               return res;
+       }
+
+       if (getcwd(wd, sizeof(wd)) == NULL) {
+               ast_test_status_update(test, "Could not determine current working directory\n");
+               ast_test_capture_free(&cap);
                return res;
        }
 
-       getcwd(wd, sizeof(wd));
        snprintf(key_dir, sizeof(key_dir), "%s/%s", wd, "tests/keys");
        push_key_dir((const char *)key_dir);
        snprintf(pub, sizeof(pub), "%s/%s.pub", key_dir, keypair1);
@@ -410,10 +428,16 @@ AST_TEST_DEFINE(crypto_verify)
 
        if (!ast_check_command_in_path(command)) {
                ast_test_status_update(test, "couldn't find %s\n", command);
+               ast_test_capture_free(&cap);
+               return res;
+       }
+
+       if (getcwd(wd, sizeof(wd)) == NULL) {
+               ast_test_status_update(test, "Could not determine current working directory\n");
+               ast_test_capture_free(&cap);
                return res;
        }
 
-       getcwd(wd, sizeof(wd));
        snprintf(key_dir, sizeof(key_dir), "%s/%s", wd, "tests/keys");
        push_key_dir((const char *)key_dir);
        snprintf(priv, sizeof(priv), "%s/%s.key", key_dir, keypair1);