>Number:         3362
>Category:       mod_auth-any
>Synopsis:       Password File is not parsed correctly
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Sat Nov  7 17:10:00 PST 1998
>Last-Modified:
>Originator:     [EMAIL PROTECTED]
>Organization:
apache
>Release:        1.2.6 and prior (and above?)
>Environment:
BSD/OS gapdev.com 3.1 BSDI BSD/OS 3.1 Virtual Kernel #12: Fri Jun 19 14:32:14 
MDT 1998

gcc version 2.7.2.1

>Description:
Password Files that are not in the format of:

  userid:password

are not parsed correctly.

This:

  userid:password:7100:100:Staranet Admin:/:ftp;mail

returns a password of:

  password:7100:100:Staranet Admin:/:ftp;mail

Anything after the first ":" is considered to be the Password.
>How-To-Repeat:
Use your /etc/passwd file for authentication or any password file that contains 
extra information after the password field.


>Fix:
authenticate_basic_user() function in mod_auth.c:

   Just before:

   /* anyone know where the prototype for crypt is? */

   if (strcmp(real_pw,(char *)crypt(sent_pw,real_pw)))


   Add:

   {
   char *real_pwptr = real_pw;

   while (*real_pwptr)
      {
      if (*real_pwptr == ':')
         {
         *real_pwptr = 0;
         break;
         }

      ++real_pwptr;
      }
   } 
>Audit-Trail:
>Unformatted:
[In order for any reply to be added to the PR database, ]
[you need to include <[EMAIL PROTECTED]> in the Cc line ]
[and leave the subject line UNCHANGED.  This is not done]
[automatically because of the potential for mail loops. ]
[If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request ]
[from a developer.                                      ]
[Reply only with text; DO NOT SEND ATTACHMENTS!         ]



Reply via email to