diff mbox series

trace-cmd: Fix warnings reported by gcc 8.2

Message ID 20181114154310.14649-1-kaslevs@vmware.com (mailing list archive)
State Superseded
Headers show
Series trace-cmd: Fix warnings reported by gcc 8.2 | expand

Commit Message

Slavomir Kaslev Nov. 14, 2018, 3:43 p.m. UTC
From: Slavomir Kaslev <kaslevs@vmware.com>

Compiling trace-cmd with gcc 8.2 reports:

    str_error_r.c: In function ‘str_error_r’:
    str_error_r.c:24:3: warning: passing argument 1 to restrict-qualified parameter aliases with argument 5 [-Wrestrict]
       snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
       ^~~~~~~~

since `snprintf` declares its first argument as __restrict but `buf` is also
passed as argument #5.

Signed-off-by: Slavomir Kaslev <kaslevs@vmware.com>
---
 lib/traceevent/str_error_r.c | 2 +-
 tracecmd/trace-read.c        | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

Comments

Slavomir Kaslev Nov. 27, 2018, 10:43 a.m. UTC | #1
Ping.
On Wed, Nov 14, 2018 at 5:43 PM <kaslevs@vmware.com> wrote:
>
> From: Slavomir Kaslev <kaslevs@vmware.com>
>
> Compiling trace-cmd with gcc 8.2 reports:
>
>     str_error_r.c: In function ‘str_error_r’:
>     str_error_r.c:24:3: warning: passing argument 1 to restrict-qualified parameter aliases with argument 5 [-Wrestrict]
>        snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
>        ^~~~~~~~
>
> since `snprintf` declares its first argument as __restrict but `buf` is also
> passed as argument #5.
>
> Signed-off-by: Slavomir Kaslev <kaslevs@vmware.com>
> ---
>  lib/traceevent/str_error_r.c | 2 +-
>  tracecmd/trace-read.c        | 8 ++++----
>  2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/lib/traceevent/str_error_r.c b/lib/traceevent/str_error_r.c
> index 503ae07..53c7206 100644
> --- a/lib/traceevent/str_error_r.c
> +++ b/lib/traceevent/str_error_r.c
> @@ -21,6 +21,6 @@ char *str_error_r(int errnum, char *buf, size_t buflen)
>  {
>         int err = strerror_r(errnum, buf, buflen);
>         if (err)
> -               snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
> +               snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d)=%d", errnum, err);
>         return buf;
>  }
> diff --git a/tracecmd/trace-read.c b/tracecmd/trace-read.c
> index c406d66..1f91fff 100644
> --- a/tracecmd/trace-read.c
> +++ b/tracecmd/trace-read.c
> @@ -386,7 +386,7 @@ static void add_pid_filter(const char *arg)
>  static char *append_pid_filter(char *curr_filter, char *pid)
>  {
>         char *filter;
> -       int len;
> +       int len, curr_len;
>
>  #define FILTER_FMT "(common_pid==" __STR ")||(pid==" __STR ")||(next_pid==" __STR ")"
>
> @@ -405,13 +405,13 @@ static char *append_pid_filter(char *curr_filter, char *pid)
>                         die("Failed to allocate for filter %s", curr_filter);
>                 sprintf(filter, ".*:" FILTER_FMT, pid, pid, pid);
>         } else {
> -
> -               len += strlen(curr_filter);
> +               curr_len = strlen(curr_filter);
> +               len += curr_len;
>
>                 filter = realloc(curr_filter, len);
>                 if (!filter)
>                         die("realloc");
> -               sprintf(filter, "%s||" FILTER_FMT, filter, pid, pid, pid);
> +               sprintf(filter + curr_len, "||" FILTER_FMT, pid, pid, pid);
>         }
>
>         return filter;
> --
> 2.19.1
>
Steven Rostedt Nov. 27, 2018, 1:58 p.m. UTC | #2
On Tue, 27 Nov 2018 10:43:17 +0000
Slavomir Kaslev <kaslevs@vmware.com> wrote:

> Ping.

Thanks for the reminder.

> On Wed, Nov 14, 2018 at 5:43 PM <kaslevs@vmware.com> wrote:
> >
> > From: Slavomir Kaslev <kaslevs@vmware.com>
> >
> > Compiling trace-cmd with gcc 8.2 reports:
> >
> >     str_error_r.c: In function ‘str_error_r’:
> >     str_error_r.c:24:3: warning: passing argument 1 to restrict-qualified parameter aliases with argument 5 [-Wrestrict]
> >        snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
> >        ^~~~~~~~
> >
> > since `snprintf` declares its first argument as __restrict but `buf` is also
> > passed as argument #5.

Ah, this is fixed in the Linux kernel tree.

Tzvetomir, can you back port commit 854e55ad289 ("objtool, perf: Fix
GCC 8 -Wrestrict error") from Linux to trace-cmd?

Thanks!

-- Steve


> >
> > Signed-off-by: Slavomir Kaslev <kaslevs@vmware.com>
> > ---
> >  lib/traceevent/str_error_r.c | 2 +-
> >  tracecmd/trace-read.c        | 8 ++++----
> >  2 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/lib/traceevent/str_error_r.c b/lib/traceevent/str_error_r.c
> > index 503ae07..53c7206 100644
> > --- a/lib/traceevent/str_error_r.c
> > +++ b/lib/traceevent/str_error_r.c
> > @@ -21,6 +21,6 @@ char *str_error_r(int errnum, char *buf, size_t buflen)
> >  {
> >         int err = strerror_r(errnum, buf, buflen);
> >         if (err)
> > -               snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
> > +               snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d)=%d", errnum, err);
> >         return buf;
> >  }
> > diff --git a/tracecmd/trace-read.c b/tracecmd/trace-read.c
> > index c406d66..1f91fff 100644
> > --- a/tracecmd/trace-read.c
> > +++ b/tracecmd/trace-read.c
> > @@ -386,7 +386,7 @@ static void add_pid_filter(const char *arg)
> >  static char *append_pid_filter(char *curr_filter, char *pid)
> >  {
> >         char *filter;
> > -       int len;
> > +       int len, curr_len;
> >
> >  #define FILTER_FMT "(common_pid==" __STR ")||(pid==" __STR ")||(next_pid==" __STR ")"
> >
> > @@ -405,13 +405,13 @@ static char *append_pid_filter(char *curr_filter, char *pid)
> >                         die("Failed to allocate for filter %s", curr_filter);
> >                 sprintf(filter, ".*:" FILTER_FMT, pid, pid, pid);
> >         } else {
> > -
> > -               len += strlen(curr_filter);
> > +               curr_len = strlen(curr_filter);
> > +               len += curr_len;
> >
> >                 filter = realloc(curr_filter, len);
> >                 if (!filter)
> >                         die("realloc");
> > -               sprintf(filter, "%s||" FILTER_FMT, filter, pid, pid, pid);
> > +               sprintf(filter + curr_len, "||" FILTER_FMT, pid, pid, pid);
> >         }
> >
> >         return filter;
> > --
> > 2.19.1
> >
diff mbox series

Patch

diff --git a/lib/traceevent/str_error_r.c b/lib/traceevent/str_error_r.c
index 503ae07..53c7206 100644
--- a/lib/traceevent/str_error_r.c
+++ b/lib/traceevent/str_error_r.c
@@ -21,6 +21,6 @@  char *str_error_r(int errnum, char *buf, size_t buflen)
 {
 	int err = strerror_r(errnum, buf, buflen);
 	if (err)
-		snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d, %p, %zd)=%d", errnum, buf, buflen, err);
+		snprintf(buf, buflen, "INTERNAL ERROR: strerror_r(%d)=%d", errnum, err);
 	return buf;
 }
diff --git a/tracecmd/trace-read.c b/tracecmd/trace-read.c
index c406d66..1f91fff 100644
--- a/tracecmd/trace-read.c
+++ b/tracecmd/trace-read.c
@@ -386,7 +386,7 @@  static void add_pid_filter(const char *arg)
 static char *append_pid_filter(char *curr_filter, char *pid)
 {
 	char *filter;
-	int len;
+	int len, curr_len;
 
 #define FILTER_FMT "(common_pid==" __STR ")||(pid==" __STR ")||(next_pid==" __STR ")"
 
@@ -405,13 +405,13 @@  static char *append_pid_filter(char *curr_filter, char *pid)
 			die("Failed to allocate for filter %s", curr_filter);
 		sprintf(filter, ".*:" FILTER_FMT, pid, pid, pid);
 	} else {
-
-		len += strlen(curr_filter);
+		curr_len = strlen(curr_filter);
+		len += curr_len;
 
 		filter = realloc(curr_filter, len);
 		if (!filter)
 			die("realloc");
-		sprintf(filter, "%s||" FILTER_FMT, filter, pid, pid, pid);
+		sprintf(filter + curr_len, "||" FILTER_FMT, pid, pid, pid);
 	}
 
 	return filter;