From 05576809194754989f88f83c7104341c35944546 Mon Sep 17 00:00:00 2001 From: Luca Boccassi Date: Fri, 21 May 2021 13:39:24 +0100 Subject: [PATCH] test-env-util: use streq_ptr when comparing directly the result of getenv It might return NULL. Fixes CID #1453247 --- src/test/test-env-util.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/test/test-env-util.c b/src/test/test-env-util.c index 1e2e9365927..ed4580e4aff 100644 --- a/src/test/test-env-util.c +++ b/src/test/test-env-util.c @@ -365,13 +365,13 @@ static void test_putenv_dup(void) { log_info("/* %s */", __func__); assert_se(putenv_dup("A=a1", true) == 0); - assert_se(streq(getenv("A"), "a1")); + assert_se(streq_ptr(getenv("A"), "a1")); assert_se(putenv_dup("A=a1", true) == 0); - assert_se(streq(getenv("A"), "a1")); + assert_se(streq_ptr(getenv("A"), "a1")); assert_se(putenv_dup("A=a2", false) == 0); - assert_se(streq(getenv("A"), "a1")); + assert_se(streq_ptr(getenv("A"), "a1")); assert_se(putenv_dup("A=a2", true) == 0); - assert_se(streq(getenv("A"), "a2")); + assert_se(streq_ptr(getenv("A"), "a2")); } static void test_setenv_systemd_exec_pid(void) { -- 2.47.3