cvs commit: apache-1.3/src/modules/experimental mod_auth_digest.c

[rse]

rse         99/11/28 04:42:01

  Modified:    src      CHANGES
               src/modules/experimental mod_auth_digest.c
  Log:
  Fixed mod_auth_digest.c: result of an open() call was being
  checked against the wrong failure value.
  
  Submitted by: Rick Ohnemus <[EMAIL PROTECTED]>
  Reviewed by: Ralf S. Engelschall
  PR: 5292
  
  Revision  Changes    Path
  1.1455    +4 -0      apache-1.3/src/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/CHANGES,v
  retrieving revision 1.1454
  retrieving revision 1.1455
  diff -u -r1.1454 -r1.1455
  --- CHANGES   1999/11/28 12:28:02     1.1454
  +++ CHANGES   1999/11/28 12:41:54     1.1455
  @@ -1,5 +1,9 @@
   Changes with Apache 1.3.10
   
  +  *) Fixed mod_auth_digest.c: result of an open() call was being 
  +     checked against the wrong failure value.
  +     [Rick Ohnemus <[EMAIL PROTECTED]>] PR#5292
  +
     *) Removed the variable name "template" from a prototype for SunOS4 
        in ap_config.h to make C++ compiler happy, too.
        [SAKAI Kiyotaka <[EMAIL PROTECTED]>] PR#5363
  
  
  
  1.11      +1 -1      apache-1.3/src/modules/experimental/mod_auth_digest.c
  
  Index: mod_auth_digest.c
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/modules/experimental/mod_auth_digest.c,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -r1.10 -r1.11
  --- mod_auth_digest.c 1999/09/19 21:28:34     1.10
  +++ mod_auth_digest.c 1999/11/28 12:41:59     1.11
  @@ -317,7 +317,7 @@
   #ifdef       DEV_RANDOM
   #define      XSTR(x) #x
   #define      STR(x)  XSTR(x)
  -    if ((rnd = open(STR(DEV_RANDOM), O_RDONLY)) == NULL) {
  +    if ((rnd = open(STR(DEV_RANDOM), O_RDONLY)) == -1) {
        ap_log_error(APLOG_MARK, APLOG_CRIT, s,
                     "Digest: Couldn't open " STR(DEV_RANDOM));
        exit(EXIT_FAILURE);