]> granicus.if.org Git - apache/commitdiff
fix or complain about some invalid errno references
authorJeff Trawick <trawick@apache.org>
Thu, 19 Nov 2009 20:00:12 +0000 (20:00 +0000)
committerJeff Trawick <trawick@apache.org>
Thu, 19 Nov 2009 20:00:12 +0000 (20:00 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@882269 13f79535-47bb-0310-9956-ffa450edef68

modules/http/http_filters.c
modules/proxy/mod_proxy_http.c
server/log.c
support/htcacheclean.c

index 01ced24944809624e4cc3b4baa5ab6d4f981cd46..befb8c6052b2f2a9c4528fa64efc781dcc5d4c68 100644 (file)
@@ -272,7 +272,6 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
             char *endstr;
 
             ctx->state = BODY_LENGTH;
-            errno = 0;
 
             /* Protects against over/underflow, non-digit chars in the
              * string (excluding leading space) (the endstr checks)
index f9018eb9eaebe3a8e19e53942d72911f19828356..93a553383256b71b732178f3ece323eb029bc5a9 100644 (file)
@@ -1145,7 +1145,7 @@ skip_body:
     }
 
     if (rv != OK) {
-        /* apr_errno value has been logged in lower level method */
+        /* apr_status_t value has been logged in lower level method */
         ap_log_error(APLOG_MARK, APLOG_ERR, 0, r->server,
                      "proxy: pass request body failed to %pI (%s)"
                      " from %s (%s)",
index 30741f7660fce32c4fab9626ae81120a74d8ae11..3733797dfdf5bf98463a474b356a1720b1b3bdaf 100644 (file)
@@ -494,6 +494,7 @@ int ap_open_logs(apr_pool_t *pconf, apr_pool_t *p /* plog */,
      * because it points to the old error log, or back to the tty
      * of the submitter.
      * XXX: This is BS - /dev/null is non-portable
+     *      errno-as-apr_status_t is also non-portable
      */
     if (replace_stderr && freopen("/dev/null", "w", stderr) == NULL) {
         ap_log_error(APLOG_MARK, APLOG_CRIT, errno, s_main,
index 756300be6f101dfef4031b196fd09cb82fb6fc3d..9991a9a4966292d579f7595797d69179036be254 100644 (file)
@@ -342,7 +342,7 @@ static int process_dir(char *path, apr_pool_t *pool)
             }
         }
 
-        /* this may look strange but apr_stat() may return errno which
+        /* this may look strange but apr_stat() may return an error which
          * is system dependent and there may be transient failures,
          * so just blindly retry for a short while
          */