On Thu, 20 Mar 2025 at 10:18, Patrick Steinhardt <ps@xxxxxx> wrote: > > On Wed, Mar 19, 2025 at 08:23:39AM +0100, Martin Ågren wrote: > > + memset(rewrap, 0, sizeof(*rewrap)); > > The `memset()` feels rather unnecessary as we only use the result at our > single caller in case we return successfully. And if we do, we know to > initialize all struct fields. Thanks, good point. I'll drop it. Martin