From af00772324533eec56696f3e0ee23136323ea5f6 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Wed, 25 Nov 2020 18:21:25 -0800 Subject: [PATCH] setlocale-tests: do not trigger gcc's -Wanalyzer-possible-null-argument * tests/test-setlocale1.c (main): Assert that each strcmp argument is non-NULL, since we don't bother handing strdup failure. --- ChangeLog | 4 ++++ tests/test-setlocale1.c | 3 +++ 2 files changed, 7 insertions(+) diff --git a/ChangeLog b/ChangeLog index bb3e664bec..49d4ed1046 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2020-11-25 Jim Meyering + setlocale-tests: do not trigger gcc's -Wanalyzer-possible-null-argument + * tests/test-setlocale1.c (main): Assert that each strcmp argument is + non-NULL, since we don't bother handing strdup failure. + raise-tests: avoid GCC 11's new exit-from-signal-handler warning gcc's -Wanalyzer-unsafe-call-within-signal-handler exposed this. * tests/test-raise.c: Include unistd.h. diff --git a/tests/test-setlocale1.c b/tests/test-setlocale1.c index 1dd41e3e8d..73c2cae375 100644 --- a/tests/test-setlocale1.c +++ b/tests/test-setlocale1.c @@ -52,6 +52,9 @@ main (int argc, char *argv[]) name2 = strdup (setlocale (LC_ALL, NULL)); + ASSERT (name1); + ASSERT (name2); + /* Test that the two results are the same. */ ASSERT (strcmp (name1, name2) == 0); free (name1); -- 2.39.5