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