From: Ruediger Pluem <rpluem@apache.org>
Date: Tue, 3 Oct 2006 12:49:42 +0000 (+0000)
Subject: Merge r452213 from trunk:
X-Git-Tag: 2.2.4~136
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cb1f0f291a45f2351eec03795f8c1a894195c10f;p=thirdparty%2Fapache%2Fhttpd.git

Merge r452213 from trunk:

* Fix precedence problem.

PR: 40658
Submitted by: Larry Cipriani <lvc lucent.com>
Reviewed by: rpluem, niq, trawick


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@452462 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index ca72796f135..2af079aa08a 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
                                                         -*- coding: utf-8 -*-
 Changes with Apache 2.2.4
 
+  *) mod_echo: Fix precedence problem in if statement. PR 40658.
+     [Larry Cipriani <lvc lucent.com>]
+
   *) mod_mime_magic: Fix precedence problem in if statement. PR 40656.
      [Larry Cipriani <lvc lucent.com>]
 
diff --git a/modules/echo/mod_echo.c b/modules/echo/mod_echo.c
index 6c8a4113c19..ddbe972c589 100644
--- a/modules/echo/mod_echo.c
+++ b/modules/echo/mod_echo.c
@@ -63,9 +63,9 @@ static int process_echo_connection(conn_rec *c)
         bb = apr_brigade_create(c->pool, c->bucket_alloc);
 
         /* Get a single line of input from the client */
-        if ((rv = ap_get_brigade(c->input_filters, bb, AP_MODE_GETLINE,
-                                 APR_BLOCK_READ, 0) != APR_SUCCESS ||
-             APR_BRIGADE_EMPTY(bb))) {
+        if (((rv = ap_get_brigade(c->input_filters, bb, AP_MODE_GETLINE,
+                                 APR_BLOCK_READ, 0)) != APR_SUCCESS) ||
+             APR_BRIGADE_EMPTY(bb)) {
             apr_brigade_destroy(bb);
             break;
         }