From 2e8a885f69afbbe4980a742580239cd007a44ae5 Mon Sep 17 00:00:00 2001 From: dtucker Date: Tue, 6 Apr 2021 09:07:33 +0000 Subject: [PATCH] Don't check return value of unsetenv(). It's part of the environment setup and not part of the actual test, and some platforms -portable runs on declare it as returning void, which prevents the test from compiling. --- regress/usr.bin/ssh/unittests/misc/test_expand.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/regress/usr.bin/ssh/unittests/misc/test_expand.c b/regress/usr.bin/ssh/unittests/misc/test_expand.c index 2a5e1bf1a5d..97804a94367 100644 --- a/regress/usr.bin/ssh/unittests/misc/test_expand.c +++ b/regress/usr.bin/ssh/unittests/misc/test_expand.c @@ -1,4 +1,4 @@ -/* $OpenBSD: test_expand.c,v 1.1 2021/03/19 03:25:01 djm Exp $ */ +/* $OpenBSD: test_expand.c,v 1.2 2021/04/06 09:07:33 dtucker Exp $ */ /* * Regress test for misc string expansion functions. * @@ -28,7 +28,7 @@ test_expand(void) TEST_START("dollar_expand"); ASSERT_INT_EQ(setenv("FOO", "bar", 1), 0); ASSERT_INT_EQ(setenv("BAR", "baz", 1), 0); - ASSERT_INT_EQ(unsetenv("BAZ"), 0); + (void)unsetenv("BAZ"); #define ASSERT_DOLLAR_EQ(x, y) do { \ char *str = dollar_expand(NULL, (x)); \ ASSERT_STRING_EQ(str, (y)); \ -- 2.20.1