HomeFreeBSD

rs: Fix a use after free.

Description

rs: Fix a use after free.

Using a pointer passed to realloc() after realloc() even for pointer
arithmetic is UB. It also breaks in practice on CHERI systems as
the updated value of 'sp' in this case would have had the bounds from
the old allocation.

This would be much cleaner if elem were a std::vector<char *>.

Reviewed by: brooks, emaste
Reported by: GCC -Wuse-after-free
Differential Revision: https://reviews.freebsd.org/D36831

Details

Provenance
jhbAuthored on Oct 5 2022, 11:47 PM
Reviewer
brooks
Differential Revision
D36831: rs: Fix a use after free.
Parents
rGbb31e1bbf2ad: rs: Fix various harmless warnings.
Branches
Unknown
Tags
Unknown