From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.0 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL autolearn=ham autolearn_force=no version=3.4.4 Received: from second.openwall.net (second.openwall.net [193.110.157.125]) by inbox.vuxu.org (Postfix) with SMTP id CBA5D23F66 for ; Mon, 6 May 2024 16:57:42 +0200 (CEST) Received: (qmail 7455 invoked by uid 550); 6 May 2024 14:57:37 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 7412 invoked from network); 6 May 2024 14:57:37 -0000 Date: Mon, 6 May 2024 10:57:50 -0400 From: Rich Felker To: lolzery wowzery Cc: musl@lists.openwall.com, Duncan Bellamy , info@bnoordhuis.nl, tony.ambardar@gmail.co Message-ID: <20240506145750.GH10433@brightrain.aerifal.cx> References: <20200119121247.37310-1-info@bnoordhuis.nl> <20220831190735.52016-1-dunk@denkimushi.com> <20220831190735.52016-2-dunk@denkimushi.com> <20240224165632.GA4163@brightrain.aerifal.cx> <20240424193038.GP4163@brightrain.aerifal.cx> <20240425122548.GQ4163@brightrain.aerifal.cx> <20240428161356.GB10433@brightrain.aerifal.cx> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240428161356.GB10433@brightrain.aerifal.cx> User-Agent: Mutt/1.5.21 (2010-09-15) Subject: Re: [musl] [PATCH 1/2] V3 resubmitting old statx patch with changes On Sun, Apr 28, 2024 at 12:13:56PM -0400, Rich Felker wrote: > On Sat, Apr 27, 2024 at 10:29:35PM -0400, lolzery wowzery wrote: > > 4. Zero all extraneous fields like __pad1 for future proofing. > > This is probably a good idea, but it should be done via > zero-initializing the whole structure before filling it, not referring > to those fields by name. The names are not a public or even > libc-internal-private interface, but placeholders, and shouldn't be > used. > > > 5. The stx_rdev_major and stx_rdev_minor fields were not correctly filled in > > Another thing I missed on the initial review. Thanks for catching it. > > > Please do not make these 5 changes yourself yet as I might find more and > > I have some great comments I want to add to explain why things are. > > If you'd like to submit the fixes, please do them as individual > changes with commit messages that explain what was wrong and what > specifically is being fixed, not a big combined "fix statx fallback" > patch. Ping. I'd like to get the uninitialized-field bugs fixed and ping distros to backport those fixes quickly so that folks don't continue generating binaries that would malfunction on older kernels. I don't really care if these are separate fixes for missing rdev, attrs, and padding, or a combined "fix uninitialized output fields in statx fallback" patch. The other things in your mails are separate and can be discussed (if there is anything wrong to begin with) without holding these known-needed fixes up. Rich