|
From: Mark W. <mj...@re...> - 2015-09-13 13:08:11
|
On Sat, 2015-09-12 at 21:37 +0200, Florian Krohm wrote:
> On 11.09.2015 17:06, Mark Wielaard wrote:
>
> > arm, kernel-4.1.6, glibc-2.22.90, gcc-5.1.1, gdb-7.10, binutils-2.25.1
>
> > arm64 kernel-4.1.5, glibc-2.22, gcc-5.1.1, gdb-7.10, binutils-2.25
>
> Ah, nice.. gcc-5.1.1.. so we can flush out format signedness issues on
> arm*
> When you get a chance, can you rerun (just make) with:
> [...]
> +format_checking_enabled=yes
> [...]
> and send me the warnings?
arm64 didn't show any -Wformat -Wformat-signedness -Wformat-security
warnings (nor any other warnings).
arm had one format warning:
m_debuginfo/readexidx.c: In function ‘vgModuleLocal_read_exidx’:
m_debuginfo/readexidx.c:941:19: warning: format ‘%lx’ expects argument
of type ‘long unsigned int’, but argument 7 has type ‘PtrdiffT {aka long
int}’ [-Wformat=]
VG_(printf)("BEGIN ML_(read_exidx) exidx_img=[%p, +%lu) "
^
There was also one -Wunused-but-set-variable warning:
m_sigframe/sigframe-arm-linux.c: In function ‘vgPlain_sigframe_destroy’:
m_sigframe/sigframe-arm-linux.c:251:27: warning: variable ‘mc’ set but
not used [-Wunused-but-set-variable]
struct vki_sigcontext *mc;
^
And lots of -Wcast-qual and -Wcast-align warnings (see attached).
Cheers,
Mark
|