Date: Mon, 7 Jul 2025 11:20:27 -0400 From: Mark Johnston <markj@freebsd.org> To: Shawn Webb <shawn.webb@hardenedbsd.org> Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 71a0af25a535 - main - libopenbsd: Add recallocarray() Message-ID: <aGvlu9Hd6VYfjM_i@nuc> In-Reply-To: <e33uojkaos2iqt2e2ogywagyi7uy6xq2u362gdn4iqhk3gosmp@ywifwaebxltw> References: <202507062318.566NIhSM031303@gitrepo.freebsd.org> <e33uojkaos2iqt2e2ogywagyi7uy6xq2u362gdn4iqhk3gosmp@ywifwaebxltw>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Jul 07, 2025 at 03:13:29PM +0000, Shawn Webb wrote: > Hey Mark, > > I've noted a couple things. > > On Sun, Jul 06, 2025 at 11:18:43PM +0000, Mark Johnston wrote: > > The branch main has been updated by markj: > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=71a0af25a5353f929521cb04c73a7266ac5b7b61 > > > > commit 71a0af25a5353f929521cb04c73a7266ac5b7b61 > > Author: Ricardo Branco <rbranco@suse.de> > > AuthorDate: 2025-06-16 21:41:48 +0000 > > Commit: Mark Johnston <markj@FreeBSD.org> > > CommitDate: 2025-07-06 23:09:00 +0000 > > > > libopenbsd: Add recallocarray() > > > > Reviewed by: kib > > MFC after: 1 month > > Pull Request: https://github.com/freebsd/freebsd-src/pull/1698 > > That appears to be the PR for POSIX O_CLOFORK. Correct. > > --- > > lib/libopenbsd/Makefile | 3 +- > > lib/libopenbsd/recallocarray.c | 82 ++++++++++++++++++++++++++++++++++++++++++ > > 2 files changed, 84 insertions(+), 1 deletion(-) > > FreeBSD commit 450dfafb1519ae7bb0615368ba45813bcabb9b35 back in 2015 > imported a different version of OpenBSD's reallocarray into libc. > Would it make sense to have only one reallocarray implementation? Or > what's the justification for having two implementations of the same > function? What might happen if someone links against both libc and > libopenbsd? Note, this is recallocarray(), not reallocarray(). libc has the latter, this commit added an implementation of the former to libopenbsd.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?aGvlu9Hd6VYfjM_i>