[PATCH] Allocation buffers for NSS result construction
DJ Delorie
dj@redhat.com
Mon May 15 23:32:00 GMT 2017
I have no specific comments on this, it looks pretty good and I like the
idea of centralizing this (I just did something similar in the NSS
testsuite, could have used this). I mostly focused on the allocation
routines; I'm not familiar with resolv yet.
I do worry about having "internal" (internal to the allocator, not
internal to glibc) functions in a header. They're written as if they'll
never be called by "outsiders" (i.e. little argument checking) but
there's no protection against such calls. I don't know how to add such
protection, though.
Not having a pointer to the buffer's beginning irked me a bit, but I
don't see any reason to include it anyway.
I would recommend a few comments defining the behavior if NULL is passed
as a thing to copy - the NSS testsuite needed to preserve such
"mistakes" in the test data.
I wonder if this new functionality is complex enough to warrant a
separate bit of documentation? Either in the manual or elsewhere...
The reallocarray patch also includes a __check_mul_overflow_size_t style
function; this duplication will need to be resolved as patches get
committed.
More information about the Libc-alpha
mailing list