This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: ntsec patch #4: passwd and group


On Wed, Nov 20, 2002 at 10:04:53AM -0500, Pierre A. Humblet wrote:
> Corinna Vinschen wrote:
> > Ahem, I thought we agreed that we don't call external functions from
> > inside Cygwin?  Never mind, there are still some of them which we have
> > to eliminate, anyway.
> 
> I didn't know about that policy but it suits me fine. As we discussed,
> internal calls to passwd/group functions should never reread the files,
> so new entry points are called for. I was going to do that in a second
> step, it wasn't a goal when I started.

The problem I'm talking about is beyond this patch.  Just reiterating
(so we know that we're talking about the same):

We talked about the need to use a static buffer in the external
getpwXXX and getgrXXX functions to allow apps to use the last return
value (a pointer) even though the file is going to be reread which
destroys all buffer space.  If Cygwin calls these functions internally,
the static buffers would be overwritten and the application itself
gets false data.

As you say, we need to substitute all internal calls to getpwXXX and
getgrXXX funcs (except getXXsid, of course) by calling appropriate
internal functions as internal_getpwuid().  Then we should be safe again.

Probably it would make sense to rename getXXsid() funcs to 
internal_getXXsid now to have a consistent naming scheme?

> How do you want to proceed? Apply this patch and undeclare internal_getpwent,
> remove LookupAccountSidA(), apply your "I'd better like" and introduce
> internal lookup functions in a few days, or prepare a single all-encompassing
> patch in a few days?

I patched your patch already slightly:

- Undeclare internal_getpwent
- "I'd better like"
- Avoid a compiler warning by using %lu instead of %u in sprintf'ing
  uids and gids.
- A few minor formatting issues.

How would you like to remove LookupAccountSidA?  Just remove it and
debug_printf ("Failed to get primary group name.") ?

Rewriting the external funcs and creating and using the internal funcs
is ok for another patch.

Corinna


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]