[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20210213190634.GC11590@brightrain.aerifal.cx>
Date: Sat, 13 Feb 2021 14:06:36 -0500
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: [PATCH v2] cuserid: support invocation with a NULL
pointer argument
On Sun, Mar 22, 2020 at 11:05:57AM -0400, Rich Felker wrote:
> On Sun, Mar 22, 2020 at 10:51:56AM +0100, Sören Tempel wrote:
> > Sören Tempel <soeren@...ren-tempel.net> wrote:
> > > From my personal Alpine Linux packaging perspective: I would prefer
> > > either removing the function entirely or supporting the invocation with
> > > a NULL pointer argument. Currently, apps using this with a NULL pointer
> > > (e.g. cdparanoia) will crash at runtime which makes it more difficult to
> > > identify them. If the function would be removed entirely we would be
> > > able to detect, during compilation, which apps use it and patch them
> > > accordingly.
> >
> > Ping.
>
> Thanks. I'm trying to catch up on patches and I just confirmed your
> research, so I think the patch is correct. I'll apply soon.
Sorry for taking so long to get back to this. The function seems to
have been significantly broken on top of not supporting a null
argument. See attached three patches which I plan to push if there are
no stupid bugs in them.
Rich
View attachment "0001-cuserid-don-t-return-truncated-results.patch" of type "text/plain" (987 bytes)
View attachment "0002-fix-misuse-of-getpwuid_r-in-cuserid.patch" of type "text/plain" (994 bytes)
View attachment "0003-fix-error-return-value-for-cuserid.patch" of type "text/plain" (1102 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.
