On Tue, Apr 16, 2024 at 04:29:32PM -0600, Todd C. Miller wrote:
> On Wed, 17 Apr 2024 00:08:30 +0200, Jeremie Courreges-Anglas wrote:
>
> > Indeed, spotted too late, thanks. But maybe the latter looks, ahem, nicer?
> > The name and ln_name buffers are sized using it.
>
> I don't think PAXPATHLEN includes space for the NUL so maybe this
> should be PAXPATHLEN+1. Or just let the sizeof() take care of it.
Well the second sizeof should take care of it, or are you suggesting
PAXPATHLEN+1+1...? :-|
char buf[sizeof("30xx linkpath=") - 1 + PAXPATHLEN + sizeof("\n")];
--
jca
