]> granicus.if.org Git - procps-ng/commitdiff
pmap: Fix minor compiler warning
authorCraig Small <csmall@dropbear.xyz>
Fri, 7 Jan 2022 20:47:46 +0000 (07:47 +1100)
committerCraig Small <csmall@dropbear.xyz>
Fri, 7 Jan 2022 20:47:46 +0000 (07:47 +1100)
/usr/include/x86_64-linux-gnu/bits/error.h:40:5: warning: ‘%s’ directive argument is null [-Wformat-overflow=]
   40 |     __error_noreturn (__status, __errnum, __format, __va_arg_pack ());
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
pmap.c: In function ‘main’:
pmap.c:760:35: note: format string is defined here
  760 |         xerrx(EXIT_FAILURE, "%s: '%s'", _("failed to parse argument"), optarg);

A simple check for optarg being null silenced this, not sure why only
this one was a problem.

Signed-off-by: Craig Small <csmall@dropbear.xyz>
pmap.c

diff --git a/pmap.c b/pmap.c
index 38b3629f3ee1c1ea06724d179b5f24ed9ec210ee..1e20fe0f5bd275a2af174b0437ad2585bfd8389c 100644 (file)
--- a/pmap.c
+++ b/pmap.c
@@ -758,7 +758,7 @@ static void range_arguments(const char *optarg)
 
        if ((buf = xstrdup(optarg)) == NULL) {
         xerrx(EXIT_FAILURE, "%s: '%s'", _("failed to parse argument"),
-              optarg);
+              (optarg?optarg:"(null)"));
     }
        arg1 = buf;
        arg2 = strchr(arg1, ',');