* [PATCH] trace-cmd: Fix warnings reported by gcc 8.2
@ 2018-11-14 15:43 kaslevs
2018-11-27 10:43 ` Slavomir Kaslev
0 siblings, 1 reply; 3+ messages in thread
From: kaslevs @ 2018-11-14 15:43 UTC (permalink / raw)
To: rostedt; +Cc: linux-trace-devel
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] trace-cmd: Fix warnings reported by gcc 8.2
2018-11-14 15:43 [PATCH] trace-cmd: Fix warnings reported by gcc 8.2 kaslevs
@ 2018-11-27 10:43 ` Slavomir Kaslev
2018-11-27 13:58 ` Steven Rostedt
0 siblings, 1 reply; 3+ messages in thread
From: Slavomir Kaslev @ 2018-11-27 10:43 UTC (permalink / raw)
To: rostedt; +Cc: linux-trace-devel
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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] trace-cmd: Fix warnings reported by gcc 8.2
2018-11-27 10:43 ` Slavomir Kaslev
@ 2018-11-27 13:58 ` Steven Rostedt
0 siblings, 0 replies; 3+ messages in thread
From: Steven Rostedt @ 2018-11-27 13:58 UTC (permalink / raw)
To: Slavomir Kaslev; +Cc: linux-trace-devel, Tzvetomir Stoyanov
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
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-28 0:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-14 15:43 [PATCH] trace-cmd: Fix warnings reported by gcc 8.2 kaslevs
2018-11-27 10:43 ` Slavomir Kaslev
2018-11-27 13:58 ` Steven Rostedt
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.