[Crash-utility] [PATCH] Add format string to *printf call
Dave Anderson
anderson at redhat.com
Tue Jul 16 19:39:25 UTC 2013
----- Original Message -----
> On 07/12/13 13:00, Stefan Bader wrote:
> > On 12.07.2013 09:16, Stefan Bader wrote:
>
> > Though this means that other architecture specific files may still have
> > issues.
>
> I just did a build on s390, s390x, ia64 and powerpc. I didn't build on ppc64
> but caught the same instance over there as well. Here are a few more that we
> missed:
>
> Signed-off-by: Troy Heber <troyh at debian.org>
> ---
Thanks Troy -- I also had gone ahead and verified everything
with real builds (except 32-bit ppc and alpha, where I used the
fake build).
Interesting side note -- while doing so, I saw that the newly-embedded
gdb-7.6 does not even build on s390, s390x and ia64 on 2.6.9-based RHEL
hosts running gcc-3.4.6. (fixable false positives)
Dave
>
> diff -urpN -urN ori/ia64.c new/ia64.c
> --- ori/ia64.c 2013-06-17 12:18:36.000000000 -0600
> +++ new/ia64.c 2013-07-16 13:25:01.111146414 -0600
> @@ -1504,7 +1504,7 @@ ia64_dis_filter(ulong vaddr, char *inbuf
> output_radix));
> if (IS_MODULE_VADDR(value) &&
> !strstr(buf2, "+"))
> - sprintf(p2, buf1);
> + sprintf(p2, "%s", buf1);
> }
> } else {
> p1 = &argv[argc-1][3];
> @@ -1515,7 +1515,7 @@ ia64_dis_filter(ulong vaddr, char *inbuf
> output_radix));
> if (IS_MODULE_VADDR(value) &&
> !strstr(buf2, "+"))
> - sprintf(p2, buf1);
> + sprintf(p2, "%s", buf1);
> }
> }
> }
> diff -urpN -urN ori/ppc.c new/ppc.c
> --- ori/ppc.c 2013-06-17 12:18:36.000000000 -0600
> +++ new/ppc.c 2013-07-16 13:25:19.582988944 -0600
> @@ -1748,7 +1748,7 @@ ppc_dis_filter(ulong vaddr, char *inbuf,
> sprintf(buf1, "0x%lx <%s>\n", value,
> value_to_symstr(value, buf2, output_radix));
>
> - sprintf(p1, buf1);
> + sprintf(p1, "%s", buf1);
> }
>
> console(" %s", inbuf);
> diff -urpN -urN ori/ppc64.c new/ppc64.c
> --- ori/ppc64.c 2013-06-17 12:18:36.000000000 -0600
> +++ new/ppc64.c 2013-07-16 13:25:19.582988944 -0600
> @@ -2387,7 +2387,7 @@ ppc64_dis_filter(ulong vaddr, char *inbu
> sprintf(buf1, "0x%lx <%s>\n", value,
> value_to_symstr(value, buf2, output_radix));
>
> - sprintf(p1, buf1);
> + sprintf(p1, "%s", buf1);
> }
>
> console(" %s", inbuf);
> diff -urpN -urN ori/s390.c new/s390.c
> --- ori/s390.c 2013-06-17 12:18:36.000000000 -0600
> +++ new/s390.c 2013-07-16 13:25:05.919105424 -0600
> @@ -1012,7 +1012,7 @@ s390_dis_filter(ulong vaddr, char *inbuf
> sprintf(buf1, "0x%lx <%s>\n", value,
> value_to_symstr(value, buf2, output_radix));
>
> - sprintf(p1, buf1);
> + sprintf(p1, "%s", buf1);
> }
>
> console(" %s", inbuf);
> diff -urpN -urN ori/s390x.c new/s390x.c
> --- ori/s390x.c 2013-06-17 12:18:36.000000000 -0600
> +++ new/s390x.c 2013-07-16 13:25:09.127078080 -0600
> @@ -1466,7 +1466,7 @@ s390x_dis_filter(ulong vaddr, char *inbu
> sprintf(buf1, "0x%lx <%s>\n", value,
> value_to_symstr(value, buf2, output_radix));
>
> - sprintf(p1, buf1);
> + sprintf(p1, "%s", buf1);
> }
>
> console(" %s", inbuf);
>
> --
> Crash-utility mailing list
> Crash-utility at redhat.com
> https://www.redhat.com/mailman/listinfo/crash-utility
>
More information about the Crash-utility
mailing list