[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cyrus-devel
Subject:    Re: [PATCH] cyrus-imapd: use getgrouplist instead of set/getgrent
From:       Tomas Janousek <tjanouse () redhat ! com>
Date:       2007-09-17 17:07:35
Message-ID: 20070917170735.GA18438 () redhat ! com
[Download RAW message or body]

Hi,

On Mon, Sep 17, 2007 at 10:15:25AM -0400, Ken Murchison wrote:
> Can you explain  the use of:
>
> if (pwd || groupids[ngroups] != gid)

Yes. It's linked to the line which reads:
> gid = pwd ? pwd->pw_gid : (gid_t) -1;

If pwd is non-NULL, we supply the gid and expect (and want) it to appear in
the list. If not, we expect -1 to appear in the list a we filter it out.

-- 
Tomas Janousek, SW Engineer, Red Hat, Inc.
[prev in list] [next in list] [prev in thread] [next in thread]