Re: CVS with PAM authentication

[Larry Jones]

Tim Kemp writes:
> 
> authentication and it all works fine!!  (Apart from cvs wouldn't create a
> new .cvspass for some reason)

A bug in the code that was fixed long ago in the development version. 
Since you seem to be comfortable applying patches, here's the fix:

Index: login.c
===================================================================
RCS file: /cvs/ccvs/src/login.c,v
retrieving revision 1.65
retrieving revision 1.66
diff -u -r1.65 -r1.66
--- login.c     4 May 2001 16:36:34 -0000       1.65
+++ login.c     16 May 2001 17:44:01 -0000      1.66
@@ -322,8 +322,8 @@
     fp = CVS_FOPEN (passfile, "r");
     if (fp == NULL)
     {
-       error (0, errno, "failed to open %s for reading", passfile);
-       goto error_exit;
+       error (0, errno, "warning: failed to open %s for reading", passfile);
+       goto process;
     }
 
     cvsroot_canonical = normalize_cvsroot (root);
@@ -360,6 +360,8 @@
            *p = '\0';
        password = xstrdup (password);
     }
+
+process:
 
     /* might as well return now */
     if (operation == password_entry_lookup)

-Larry Jones

Moms and reason are like oil and water. -- Calvin