linux arm regtest: fix memcheck leak-segv-jmp

The hand rolled asm syscall to mprotect was failing.
At least it now passes on my rpi.

lnux helgrind: broaden suppression in libnss_mdns4

linux arm regtest: fix test and stdout expected

The three diffs that I noticed earlier were really due to typos
in the testcase. They should have been using d0 d1 d2 d3 but
instead were using d0 d1 d2 d4.

Update README_PACKAGERS

Give another example of why stripping the binaries is a bad idea.

FreeBSD: add suppressions for reachable memory in libthr.so

Fix gdbserver tests failing due to 'warning: ' prefix in syscall-template.S line

At least on debina 12.5, with GDB 13.1 and GDB 14.1, 2 gdbserver tests
fails due to a 'warning: ' word appearing in from of a line
giving the syscall-template.S filename.

Fix this by allowing an optional 'warning: ' regexp at the beginning of the line.

Tested on debian 12.5, with GDB 12.1/13.1/14.1 and 15.0.50.20240211-git

FreeBSD regtest: improve memcheck filtering for varinfo5

Now only one real difference between clang and GCC.

FreeBSD regtest: updates for memcheck varinfo5

Two things:
Improvement in reading debuginfo for variables.
clang16 doing tail call optimization, so add -fno-optimize-sibling-calls

Bug 471036 - Add test for rorx with rip-relative address

Bug 471036 - Print more context for amd64 disasm rip mismatch

Bug 471036 - disInstr_AMD64: disInstr miscalculated next %rip on RORX imm8, m32/64, r32/6

Patch contributed by redoste@redoste.xyz

NEWS: add bug 481131

x86 regtest: fix clobber lists in generated asm statements

Must add result registers also, or else code like this is generated:

      asm(
         "fsave %3\n"
         "movl %2, %%eax\n"
         "cdq\n"
         "movl %%edx, %0\n"
         "movl %%eax, %1\n"
         "frstor %3\n"
         "cld\n"
         : "=m" (result0), "=m" (result1)
         : "m" (preset0), "m" (state[0])
         : "eax"
      );

This crashes on my machine because gcc places some pointer in %%edx.

manpage: a few changes for realloc size zero and give FreeBSD a few mentions

command line help: move --realloc-zero-bytes-frees

It should be in the section on tools that replace malloc.

linux arm regtest: update neon64 expected

I get the same output from both Valgrind and running the test
natively.

Bug 283429 - ARM leak checking needs CLEAR_CALLER_SAVED_REGS

Unfortunately it took over 12 years to reinvent the wheel and
fix this item.

See also
d91042c29dd3fbe2971f36f35b08e9735327f50a
and
b0015f7572bf42ee4ac1296884f5e7640ec1e5dc

FreeBSD _umtx_op: fix bad wrapper for robust lists

Was a half-baked copy and paste from the mutex version.

Add missing gitignore for none arm64 ldxp_stxp

Typo in comment

Add myself to AUTHORS

linux aarch64: an extra Helgrind printf suppression and dix a couple of warnings

The existing suppression begins with

   fun:memcpy*

but I'm seeing the first one as

   fun:__GI_memcpy

so I just made that *memcpy*

linux regtest: add prereq for bug480706

For systems without openssl

FreeBSD regtest: feature test for aio_readv

Doesn't exist on FreeBSD 12.4 and earlier.
Also clean up configure output - was missing a few yes/nos.

Linux regtest: link for bug480706

Needed to use LDADD not LDFLAGS

Linux PPC32: fix typo

Full stop instead of a comma.

Bug 480706 - Unhandled syscall 325 (mlock2)

FreeBSD: updates for FreeBSD 14.0

Added some filtering. Removed a few compiler warnings.
memcheck manuel1 has an extra error somewhere within
the printf calltree - sensitive to libc changes.

I still get
== 853 tests, 2 stderr failures, 0 stdout failures, 0 stderrB failures, 0 stdoutB failures, 0 post failures ==
memcheck/tests/descr_belowsp             (stderr)
memcheck/tests/varinfo5                  (stderr)

descr_belowsp no longer mentions the guard page. Back in commit

121374b287ce39ef0e42e45dfc27ae0949e6f528

I said about FreeBSD 14 adding ASLR

"    Some more work will be needed for the stack size."

Looks like that has now come home to roost.

varinfo5 has lost some inline info but gained info on variable names.

Bug 476331 - clean up generated/distributed filter scripts

Makefiles: bug 476331 cleanup

Quick test with one script to see if buildbot is happy.

regtest: some missing AM_CFLAGS and AM_CXXFLAGS

In all cases, these caused problems for 32on64 builds.

In a few cases extra CFLAGS were needed but AM_CFLAGS not used.

And it seems as though autoconf needs explicit else branches. So

if FOO
test_CFLAGS = ${AM_CFLAGS} -hide-some-warnings
endif

doesn't set test_CFLAGS to the default when FOO is false.
(I'm sure I've seen that before, and forgotten).

Linux PPC: fix the constant used for O_DIRECT

There are two defined

$ grep -rI O_DIRECT /usr/include/* | grep -v ORY | grep -v __
/usr/include/asm/fcntl.h:#define O_DIRECT       0400000 /* direct disk access hint */
/usr/include/asm-generic/fcntl.h:#define O_DIRECT       00040000        /* direct disk access hint */

and of course I used the wrong one. The non-generic value is the right one.

Bug 480405 - valgrind 3.22.0 "m_debuginfo/image.c:586 (set_CEnt): Assertion '!sr_isError(sr)' failed."

Bug 480488 - Add support for FreeBSD 13.3

FreeBSD ioctl: add handling for FIODGNAME

Up til now FreeBSD ioctl coverage was conspicuous by its absence
relying entirely on the generic wrapper. Time to start improving
that.

Example provided by Kyle Evans.

Revert compiler flags for armv7

ARM: handle v6 to v8 in configure.ac

Some changes based on first patch in
https://bugs.kde.org/show_bug.cgi?id=276897
Bug 276897 ARM v6 legacy patches
contributed by "JW"

I've also modified the GCC flags for a few tests in none/tests/arm

We probably need more arm feature tests in configure.ac.

DRD regtest: add ARM expecteds for annotate_trace_memory

The diff is that one 8byte access is being split into two 4byte
acceses.

FreeBSD regtest: make memcheck varinfo3 filtering consistent

Same formar for static_local_def

memcheck regtest arm: add a clobber to r0

This fixes a few of the leak testcases

regtest gdbserver: add an extra filter for Raspberry Pi OS select

Helgrind: add suppression for libnss from getaddrinfo

Seems as though Raspberry Pi OS doesn't have a debuginfo package
for libnss_mdns4 so we need to suppress based on the obj name.

There is still one error in the Helgrind getaddrinfo test, but that
is in the somewhat malformed libarmmem library (which does have
debuginfo but Valgrind can't read it because there is no ELF type
or size info). I think that suppressing that would be too broad.

gdbserver_tests/filter_gdb.in: delete thread exiting

The hginfo testcase would fail with GDB 14.1 because it would output
an extra line saying: [Thread .... exited]

Bug 480126 - Build failure on Raspberry Pi 5 / OS 6.1.0-rpi7-rpi-v8

The problem is that the Pi 3+ with raspbian has a 64bit aarch64 OS
but a 32bit gnueabihf userland. If you just run "configure" the
result will be an attempt to use 32-bit GCC with the ARM64 Valgrind
platform, which will fail.

This can be fixed by using the appropriate --host= option.
I've updated README.aarch64 with a description of this.

I've also changed configure.ac to take "arm*" for the
ARM platform (previously it was "armv7*"). That's been
a patch on Debian for a while. It will allow building
on older arm hardware and also armv8 (used on the
Raspberry Pi 3 and later models).

Bug 480052 - WARNING: unhandled amd64-freebsd syscall: 580

FreeBSD: revert a change for debugging ifunctions and correct a couple of typos

regtest: try to make cachegrind/tests/ann-merge2.vgtest deterministic

Add a touch of the cgout files so that they are more recent than the
source file. git clone seems to sometimes timestamp the source
after the cgout files which generates a warning and a post failure.
At least with FreeBSD on ZFS.

Fix unused variable warning

Darwin: Fix another bad merge of coregrind/m_debuginfo/debuginfo.c

Darwin: Fix bad merge of coregrind/m_debuginfo/debuginfo.c

Add Bug 479041 to NEWS

Bug 479041 - Executables without RW sections do not trigger debuginfo reading

The main change is to not assume that there is always 1 and only 1 RW segment.
Now the rw segment count is obtained from the macho segments.

I've had to make several changes to remove asserts that checked
that there is always 1 or more rw segments. I don't think that this
really affects 'normal' C and C++ compiled binaries. There is one
exp-bbv testcase (x86/million) that is written in assembler and
was failing until I removed all of the asserts.

There's still a bit more work to do.
1. Handle fat binaries - are these still a thing (with "apple silicon")?
2. Use a dynamically sized buffer for the segments rather than just 4k.

Illumos regtest: memmem testacase update

Like Darwin, Illumos memmem returns NULL if either length is 0.

Darwin regtest: modify filter for massif

Now only have 3 failing testcases with 2 causes
- brk() not handled
- not redirecting alloc functions in the main exe

Darwin massif: add lots of ignore-fn for the many dynamic allocations done at startup

mips: add missing header for nanomips

Add the missing header and fix nanomips build of Valgrind.

Darwin regtest: try again with memmem and make sigreturn warnings debug level 1

Darwin regtest: memmem returns NULL if either length is 0.

FreeBSD regtest: broaden unknown suppression

Also add comment about recently obsoleted syscalls

FreeBSD regtest: ensure aio_read string is null terminated

Was failing on FreeBSD 14 x86

linux regtest: remove alloc-size-larger-than warning for reallocarray

linux regtest: turn off stringop-overflow  warning for strlcat_strlcpy

Skip updating rodata avma and tracing when there are non-contiguous rodata sections.

Bug 478837 - valgrind fails to read debug info for rust binaries

mold produces binaries with non-contiguous .rodata and .rodata.xxx
sections. The code that merges their addresses can only handle 1
rodata which means only contiguous sections can be merged.

Since this now seems to be a "normal" case I've changed the warning
message to be a SYMTAB_TRACE message.

Bug 478624 - Valgrind incompatibility with binutils-2.42 on x86 with new nop patterns (unhandled instruction bytes: 0x2E 0x8D 0xB4 0x26)

It was a bit of a struggle to get the testcase to build
with both clang and gcc (oddly enough gcc was more difficult) so
I just resorted to using .byte arrays.

Linux: broaden one of the dynamic thread vector reachable suppressions

On Arch Linux the callstack has two calls to malloc whilst the old
suppression only had one.

FreeBSD: another reachable suppression, for getgrnam this time

FreeBSD: add suppressions for still reachable memory when using getaddrinfo

Also add a couple of attributions for my recent memccpy changes.

Add a redir for memccpy

Add bug 478211 to NEWS

Commit ec73dcb18 remote-utils-shared: Created files for sharing code

remote-utils-shared: Created files for sharing code

between vgdb.c and m_gdbserver/server.c. This removes code
redundancies that was introduced for vgdb.c by vgdb --multi
implementation.

coverity: mostly deadcode warnings that depend on platform macros

So just turn them off

Add fchmodat2 syscall on linux

fchmodat2 is a new syscall on linux 6.6. It is a variant of fchmodat
that takes an extra flags argument.

https://bugs.kde.org/show_bug.cgi?id=477198

vgdb.c: Encode the qRcmd reply as hex string

When vgdb replies to qRcmd packet recieved from GDB, it
used to send the reply as simple string. This is wrong,
GDB expects to get a hex encoded string.

https://bugs.kde.org/show_bug.cgi?id=477719

coverity: deadcode and unsigned comparison with 0 warnings

(really) Bug 477630 - Include ucontext.h rather than sys/ucontext.h in Solaris sources

Mixed up my patches and the previous commit should have been
Bug 477628 - Add mremap support for Solaris

Patch provided by
   Jakub Kulik kulikjak@gmail.com

Bug 477630 - Include ucontext.h rather than sys/ucontext.h in Solaris sources

Patch provided by
   Jakub Kulik kulikjak@gmail.com

coverity: a few more unsigned comparisons and unused return values

launcher-linux.c (find_client): barf if we don't have read permission for executable

For such an executable valgrind will now report:

valgrind: Cannot continue: Need read permission on /usr/local/bin/exec-no-read

https://bugs.kde.org/show_bug.cgi?id=281059

Darwin: clean up 3 warnings

FreeBSD regtest: make aio test more robust

Had to remove a few of the errors - seems to me that aio errors
are not handled well by the FreeBSD kernel. E.g., it looks
like the error state doesn't get cleared by aio_return.

FreeBSD regtest: new aio test sometimes hangs in infinite aio_error loop

Put a limit on the loop. Need to check if this is not a problem with
aio_read corrupting memory.

Darwin: add sigwait syscall wrapper

The FreeBSD testcase doesn't work, but the problem seems
to be with the initial signal()/raise(). Needs more work.

coverity: unsigned comparisons with 0

FreeBSD: wrong level of indirection and unsigned >= 0 comparison

coverity: unreachable and unused

arm64: remove a couple of lines of dead code

mpi: remove NULL checks before frees

As Coverity points out, paths reaching these NULL checks will
have already dereferenced the pointers. So use the same condition
as for the allocation.

Darwin: another csrctl change

Still getting asserts on x86 probably due to 64 bit arguments
(none of the Darwin syscalls are split into x86 / amd64 with
MERGEs for x86).

Darwin: missing defines to go with previous change

Darwin: try to fix csrctl syscall wrapper

memcheck/tests/x86/xor-undef fails with
+valgrind: m_syswrap/syswrap-darwin.c:9810 (void vgSysWrap_darwin_csrctl_before(ThreadId, SyscallArgLayout *, SyscallArgs *, SyscallStatus *, UWord *)): Assertion 'sizeof(user_addr_t) <= sizeof(RegWord)' failed.
(on macOS 10.13)

Change copied from
   https://github.com/LouisBrunner/valgrind-macos

Darwin regtest: add testcase for bug390269

Darwin: openat(_nocancel) fixes

One wrong warning string and missing macro declarations

Darwin: add suppressions for a couple of crypto functions

Darwin: add missing define for VKI_AT_FDCWD

Bug 390269 - unhandled amd64-darwin syscall: unix:464 (openat_nocancel)

Copied from https://github.com/LouisBrunner/valgrind-macos.git

Regtest to come shortly

Bug 428364 - Signals inside io_uring_enter not handled

Patch provide by
    Andrew Soutar kdebugtraq@andrewsoutar.com
Thanks!

I haven't added the reproducers as they require interrupting
the Valgrind process. Maybe the tests can be adapted to use
fork and have the parent kill the child running io_uring.

linux helgrind: broaden __printf_buffer_write suppression

On Fedora 39 pth_mempcpy_false_races was failing because
the callstack now has an extra inline and memcpy is versioned

Bug 466884 - Missing writev uninit padding suppression for _XSend

Bug 476548 - valgrind 3.22.0 fails on assertion when loading debuginfo file produced by mold