Subject: Re: chmod patch
To: Christos Zoulas <firstname.lastname@example.org>
From: Liam J. Foy <email@example.com>
Date: 05/17/2005 20:53:28
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.
I think you may be wrong here:
if ((set = malloc((u_int)(sizeof(BITCMD) * setlen))) == NULL)
saveset = set;
endset = set + (setlen - 2);
As you can see, it will return before we hit anything else to set errno.
- Liam J. Foy