Mailing-List: contact cygwin-developers-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT cygwin DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT cygwin DOT com>
List-Help: <mailto:cygwin-developers-help AT cygwin DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-developers-owner AT cygwin DOT com
Delivered-To: mailing list cygwin-developers AT cygwin DOT com
Message-ID: <3D36EF6B.E6916EC8@ieee.org>
Date: Thu, 18 Jul 2002 12:40:11 -0400
From: "Pierre A. Humblet" <Pierre DOT Humblet AT ieee DOT org>
X-Accept-Language: en,pdf
MIME-Version: 1.0
To: cygwin-developers AT cygwin DOT com
Subject: Re: Corinna or Pierre please comment? [jason AT tishler DOT net: Re: setuid() 
 problem when disconnected from PDC under 1.3.12-2]
References: <20020715155951 DOT GG2372 AT tishler DOT net> <3 DOT 0 DOT 5 DOT 32 DOT 20020715162535 DOT 0080c900 AT mail DOT attbi DOT com> <20020716191735 DOT GB1692 AT tishler DOT net> <20020717121704 DOT GA1260 AT tishler DOT net> <20020717142259 DOT E6932 AT cygbert DOT vinschen DOT de> <20020717123351 DOT GB1260 AT tishler DOT net> <20020717145642 DOT G6932 AT cygbert DOT vinschen DOT de> <3D358707 DOT 509E8CD3 AT ieee DOT org> <20020717163922 DOT GD1260 AT tishler DOT net> <3 DOT 0 DOT 5 DOT 32 DOT 20020717222627 DOT 007fd3e0 AT mail DOT attbi DOT com> <20020718105632 DOT R6932 AT cygbert DOT vinschen DOT de>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Corinna Vinschen wrote:

> What do you mean?  A debug output when NetGetDCName() fails is already
> in the code and when get_user_groups() fails, too.  What does an extra
> system_printf help?

The debug output is only visible with strace. Somehow I would like to convey 
to unsophisticated users that they should edit /etc/group if they have 
a permission problem when running disconnected. That should be written
in some doc...
 
> Could you just hand in the ChangeLog?  I will apply the patch ASAP then.

That will be for tonight. There will be a slightly improved patch, 
avoiding duplicate groups.

By the way, 
   struct __group32 *gr;
   if (gr->gr_gid == pw->pw_gid)
 causes a compiler warning about signed/unsigned comparison.
 struct passwd has an "int pw_gid" field. Is that posix?
 
I am also working on a setgroup().

Pierre