You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by wr...@locus.apache.org on 2000/12/13 06:06:44 UTC

cvs commit: apache-1.3/src/modules/standard mod_status.c

wrowe       00/12/12 21:06:43

  Modified:    src      CHANGES
               src/modules/standard mod_status.c
  Log:
    mod_status now tests the refresh value, and will only pass values for
    ?refresh= that are 1 or greater.  Otherwise defaults to 1 (per ?refresh).
  
  PR: 5067
  Submitted by:	Dirk Ahlers <th...@darkride.net>
  Reviewed by:	William Rowe
  
  Revision  Changes    Path
  1.1605    +4 -0      apache-1.3/src/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/CHANGES,v
  retrieving revision 1.1604
  retrieving revision 1.1605
  diff -u -r1.1604 -r1.1605
  --- CHANGES	2000/12/12 06:56:54	1.1604
  +++ CHANGES	2000/12/13 05:06:41	1.1605
  @@ -1,5 +1,9 @@
   Changes with Apache 1.3.15
   
  +  *) mod_status now respects ?refresh=n of 1 or greater.  If the given
  +     refresh value is not a number, ?refresh is set to 1 second.
  +     [William Rowe, Dirk Ahlers <th...@darkride.net> PR5067]
  +
     *) While Windows 95 and 98 may still run Apache as a 'service' in a 
        hidden window, the Apache Win9x service now dies when the user 
        logs off.  No immediate or obvious solution available, but this
  
  
  
  1.116     +3 -1      apache-1.3/src/modules/standard/mod_status.c
  
  Index: mod_status.c
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/modules/standard/mod_status.c,v
  retrieving revision 1.115
  retrieving revision 1.116
  diff -u -r1.115 -r1.116
  --- mod_status.c	2000/11/14 09:57:28	1.115
  +++ mod_status.c	2000/12/13 05:06:42	1.116
  @@ -274,7 +274,9 @@
   	    if ((loc = strstr(r->args, status_options[i].form_data_str)) != NULL) {
   		switch (status_options[i].id) {
   		case STAT_OPT_REFRESH:
  -		    if (*(loc + strlen(status_options[i].form_data_str)) == '=')
  +		    if (*(loc + strlen(status_options[i].form_data_str)) == '='
  +                        && (loc + strlen(status_options[i].form_data_str) 
  +                                    + 1) > 0)
   			ap_table_set(r->headers_out,
   			      status_options[i].hdr_out_str,
   			      loc + strlen(status_options[i].hdr_out_str) + 1);