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

FreeBSD: make warnings about casting away volatile go away

freebsd: unused variable warning

still not sure what to do in aio_cancel

darwin: syswrap-darwin.c warning cleanup

Darwin: remove global variable used in aio_return

This was using a global variable to signify a read success
between pre- and post- aio_return. I don't think that this
was safe. There could be multiple calls to aio_return on
different threads. The global aiocpb_table is safe enough
as the pointers to control block are unique. However if two
threads manipulate was_a_successful_aio_read there's no way
of telling them apart. Rather than trying to make
was_a_successful_aio_read thread safe I did the same as on
FreeBSD and removed post- aio_return and now do everything
in pre- aio_return. No global variable, no thread hazard.

FreeBSD regtest: unused variable warning

And ensure that the output string is always terminated.

syswrap renameat2 on ppc32, mips32 and mips64 linux

syswrap preadv2 and pwritev2 also on ppc32-linux

s390x-linux also has memfd_secret, syswrap it

syswrap openat2 for all linux arches

support pidfd_getfd

Signed-off-by: Ali Polatel <alip@chesswob.org>

valgrind-monitor.py regular expressions should use raw strings

With python 3.12 gdb will produce the following SyntaxWarning when
loading valgrind-monitor-def.py:

  /usr/share/gdb/auto-load/valgrind-monitor-def.py:214:
  SyntaxWarning: invalid escape sequence '\['
    if re.fullmatch("^0x[0123456789ABCDEFabcdef]+\[[^\[\]]+\]$", arg_str):

In a future python version this will become an SyntaxError.

Use a raw strings for the regular expression.

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

freebsd regtest: update expected

hadn't rebuilt the test exe after changes

Bug 476887 - WARNING: unhandled amd64-freebsd syscall: 578

Adds syscall wrappers for aio_readv and aio_writev
Also rewrote the wrappers for aio_read, aio_write and aio_return
as they weren't correctly checking the async memory.

The code is similar th that of Darwin with one exception.
Darwin uses a global variable to communicate between the
pre- and post- aio_return wrappers. I don't think that is
safe when there are multiple aio reads in flight. Instead
I put everything in the pre- aio_return wrapper.

Fix memcheck/tests/darwin/Makefile.am EXTRA_DIST typo

bug437790.stderr.out should have been bug437790.stdout.exp

Bug 437790 - valgrind reports "Conditional jump or move depends on uninitialised value" in memchr of macOS 10.12-10.15

linux regtest: typo in EXTRA_DIST

Bug 476780 - Extend strlcat and strlcpy wrappers to GNU libc (part 2)

Linux regtest

Bug 476780 - Extend strlcat and strlcpy wrappers to GNU libc

Will add Linux regtest soon

Bug 476787 - Build of Valgrind 3.21.0 fails when SOLARIS_PT_SUNDWTRACE_THRP is defined

Patch provided by
   Jakub Kulik kulikjak@gmail.com

Bug 460616 - Add support for aarch64 dotprod instructions

This change adds support for the FEAT_DotProd instructions
SDOT <Vd>.<Ta>, <Vn>.<Tb>, <Vm>.4B[<index>]
SDOT <Vd>.<Ta>, <Vn>.<Tb>, <Vm>.<Tb>
UDOT <Vd>.<Ta>, <Vn>.<Tb>, <Vm>.4B[<index>]
UDOT <Vd>.<Ta>, <Vn>.<Tb>, <Vm>.<Tb>

regtest: bug401284.c should never cast the return of malloc in C

filter out dl-start.S: No such file or directory in gdbserver_tests

On some aarch64 GNU/Linux systems gdb will be unable to find the
dl-start.S file and complain. Filter out this message to make the
tests pass.

Bug 401284 - False positive "Source and destination overlap in strncat"

Need to also look at strlcat and plain strcat.

regtest: add a deliberately false address to drd and helgrind getaddrinfo

Make sure that at least one call fails.

linux regtest: reallocarray was only added in glibc 2.29

Add a feature check

Bug 475498 - Add reallocarray wrapper

massif regtest: add an existence check to overloaded-new

For old old compilers that don't have aligned new

massif regtest: add 'pool' to ignore functions for Arch Linux

massif regtest: typo in Makefile.am

Bug 476535 - Difference in allocation size for massif/tests/overloaded-new between clang++/libc++ and g++/libstdc++

In the end all I could do was filter the results.

libc++ and libstdc++ allocate different sizes of stuff for their own usr.
That means that when we get to allocating aligned blocks there is some
slop (up to the alignment size) that gets counted. And the amount of
that slop depends on the prior (internal) allocations.

massif regtest: stray space in expected

massif regtest: make verbose tests work on both 32bit and 64bit

Now alloc fns only have the appropriate new overloads which
means 32bit has unsigned size but 64bit has unsigned long.
Filter the unsigned long to unsigned.

massif regtest: update aloc fns in verbose expecteds

massif regtest: try adding a new expected for overloaded-new

Added a filter for "pool" that's used on Arch Linux
I'm not too confident that the new expected will work for
all architectures or even versions of libstdc++

massif: clean up alloc functions

Missed out aligned_alloc
Left in the alignment argument name and didn't use east const for
nothrow.

massif regtest: keep churning with overloaded-new test

Solaris 11.3 doesn't have aligned_alloc so remove it.

In any case I think it's better to use dummy allocators since
that means there is no risk of accidentally falling through
to malloc/free.

massif regtest: use the correct aligned delete overloads

Still trying to see why libstdc++ and libc++ allocate different amounts

massif regtest: remove overloaded-new.post.exp-freebsd

There was a duplicate at the end of the list

massif regtest: fix building aligned new overloads with pre-C++17 compilers

FreeBSD: remove some massif regtest expecteds

These were added mainly to have a cleanish slate with GCC on FreeBSD.
Unfortunately FreeBSD doesn't have dependent packages, so no debuginfo
for libstdc++ and the GCC port libstdc++ is stripped. That means that
the filters for internal GCC allocations don't work.

Since I hardly use GCC any more I'm getting rid of these expecteds.

massif: add C++ aligned operator new to allocator functions

Also make the unsigned (32bit) and unsigned long (64bit) for size_t
conditional on the wordsize.

Do not run memcheck/tests/x86-linux/scalar when root

The test expects some syscalls to fail, but when run as root
the user actually has permission to do silly things. So skip
the test if we notice we are root.

Non-FreeBSD regtest: fix build of memcheck/tests/x86/pcmpgt

As suggested by Mark Wielaard. It looks like AM conditionals
behave as though they have an implicit, empty, else.