Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220515183840.csmpbq36vd3mmhan@eve>
Date: Sun, 15 May 2022 18:38:40 +0000
From: Alyssa Ross <hi@...ssa.is>
To: Rich Felker <dalias@...c.org>
Cc: musl@...ts.openwall.com
Subject: Re: [PATCH musl v2 3/3] mntent: fix parsing lines with
 optional fields

Hi Rich,

On Thu, May 12, 2022 at 02:15:53PM -0400, Rich Felker wrote:
> Alright. It sounds like your patch is pretty much okay as-is except
> for the added " || n[1]==len" in the while condition and removing the
> (now unreachable) return-zero condition at the end. If that sounds
> right and it's okay with you I'm happy to commit your existing patch
> with those changes then make any other improvements myself or review
> further patches from you separately if you like. What do you think?

Sounds good.  Please go ahead and do it yourself —-it would take me a
fair bit of time to put my Musl development setup back together, so if
you're happy to fix up and commit the patch there's no point in me
doing that.

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.