Larry,

Hi! I changed the source according to your suggestion and the following happened
 ->

<unixprompt>$cvs status -v readme.txt
setgid failed: Not owner
cvs status: authorization failed: server <servername> rejected access

Please let me know if you need more information about my setup

I think i dont have the latest version of the server.c file (as I had to
manually make the change on line 5084 instead of 5114 in your patch).
How do I know what revision of server.c i am using? How do i get the latest cvs
tarball with any patches? I've got cvs 1.10.8

Thanks heaps,
aditya

P.S. -> Making sure i've got it right...
on AIX, the initgroups(3) system call, prints a message on stderr and return an
error when the underlying call to setgroups fails.

Larry Jones wrote ->
Whoever wrote the initgroups(3) function decided that
it wasn't sufficient to just return an error if the underlying call to
setgroups fails, he or she decided to print a message to stderr as well!
This, of course, is a complete travesty.  If you have source code, it's
easily fixed; in either case, you should submit a bug report to whoever
maintains your OS.






[EMAIL PROTECTED] on 05/05/2000 06:17:07
                                                              
                                                              
                                                              
 To:      [EMAIL PROTECTED]                       
                                                              
 cc:      Aditya Sanghi/Net@Saville, [EMAIL PROTECTED]         
                                                              
                                                              
                                                              
 Subject: Re: pserver problem in moving from 1.10 to 1.10.8   
                                                              







Larry Jones writes:
>
> I strongly suggest kicking the IBM machinery so that it eventually gets
> fixed, but in the meantime, you can use the following (unofficial and
> unlikely to be adopted into the main source tree) patch:

Sorry, that escaped prematurely.  Here's the patch:

diff -u -r1.227 server.c
--- server.c    2000/05/04 19:21:10     1.227
+++ server.c    2000/05/04 19:22:09
@@ -5114,7 +5114,7 @@
     }

 #if HAVE_INITGROUPS
-    if (initgroups (pw->pw_name, pw->pw_gid) < 0
+    if (geteuid() == 0 && initgroups (pw->pw_name, pw->pw_gid) < 0
 #  ifdef EPERM
        /* At least on the system I tried, initgroups() only works as root.
           But we do still want to report ENOMEM and whatever other

-Larry Jones

Another casualty of applied metaphysics. -- Hobbes



Reply via email to