From: Manoj Kasichainula <manoj@apache.org>
Date: Tue, 26 Oct 1999 22:25:02 +0000 (+0000)
Subject: ap_bflush and ap_bclose now return ap_status_t error codes instead of
X-Git-Tag: 1.3.10~221
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8c4d317e7fb331e52ecec10028302b51d599f596;p=apache

ap_bflush and ap_bclose now return ap_status_t error codes instead of
returning -1 and setting errno.


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@84040 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/http/http_protocol.c b/modules/http/http_protocol.c
index 39e82db683..cdaec1985c 100644
--- a/modules/http/http_protocol.c
+++ b/modules/http/http_protocol.c
@@ -2339,13 +2339,16 @@ API_EXPORT_NONSTD(int) ap_rvputs(request_rec *r,...)
 
 API_EXPORT(int) ap_rflush(request_rec *r)
 {
-    if (ap_bflush(r->connection->client) < 0) {
+    ap_status_t rv;
+
+    if ((rv = ap_bflush(r->connection->client)) != APR_SUCCESS) {
         if (!ap_is_aborted(r->connection)) {
-            ap_log_rerror(APLOG_MARK, APLOG_INFO, errno, r,
+            ap_log_rerror(APLOG_MARK, APLOG_INFO, rv, r,
                 "client stopped connection before rflush completed");
             ap_bsetflag(r->connection->client, B_EOUT, 1);
             r->connection->aborted = 1;
         }
+        errno = rv;
         return EOF;
     }
     return 0;
diff --git a/server/connection.c b/server/connection.c
index 6f740f367a..de55aba4f4 100644
--- a/server/connection.c
+++ b/server/connection.c
@@ -146,7 +146,7 @@ static void lingering_close(request_rec *r)
 
     /* Send any leftover data to the client, but never try to again */
 
-    if (ap_bflush(r->connection->client) == -1) {
+    if (ap_bflush(r->connection->client) != APR_SUCCESS) {
 	ap_bclose(r->connection->client);
 	return;
     }