Subject: Re: chmod patch
To: Christos Zoulas <firstname.lastname@example.org>
From: Liam J. Foy <email@example.com>
Date: 05/17/2005 21:17:09
On Tue(17)/May/05 - , Christos Zoulas wrote:
> In article <20050517183626.GA468@anarion>,
> Liam J. Foy <firstname.lastname@example.org> wrote:
> > Could you guys look at the following patch:
> >- setmode(3) can return NULL due to malloc failing. However,
> > if malloc does fail an incorrect error message would be
> > printed. You can resolve this by checking errno.
> Actually someone will need to fix the source for setmode in order
> for your patch to work. There are other calls that can set errno
> (strtol() for example), so your patch is incorrect. You should
> file a PR about fixing error checking and documentation in setmode.
Hmm, I see now. Maybe we could check for ENOMEM?
- Liam J. Foy