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...@apache.org on 2005/08/09 16:54:50 UTC

svn commit: r231046 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

Author: wrowe
Date: Tue Aug  9 07:54:46 2005
New Revision: 231046

URL: http://svn.apache.org/viewcvs?rev=231046&view=rev
Log:

  Hopefully, address the last edge case where status may 
  be uninitialized.  Asserts in non-debug builds are bad things,
  anyways, so this is probably more correct.  This should fix
  -Werror compile warning observed by Joe Orton.

Modified:
    httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/proxy/mod_proxy_http.c?rev=231046&r1=231045&r2=231046&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_http.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_http.c Tue Aug  9 07:54:46 2005
@@ -949,7 +949,8 @@
                                               || (bytes_read > 0));
         break;
     default:
-        ap_assert(1 != 1);
+        /* shouldn't be possible */
+        status = APR_EINVAL;
         break;
     }