nicer to compare r->finfo.filetype with APR_NOFILE instead of 0

the sub request output filter shouldn't lose the return code from
the next filter

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

diff --git a/server/request.c b/server/request.c
index b015def039f33ba8edff8fb9f224d666b09e33bf..4e2e1edbc58434777b3fcef1c7eb790ea0bbb0db 100644 (file)
--- a/server/request.c
+++ b/server/request.c
@@ -231,7 +231,7 @@ static int get_path_info(request_rec *r)
     char bStripSlash=1;
 #endif
 
-    if (r->finfo.filetype) {
+    if (r->finfo.filetype != APR_NOFILE) {
        /* assume path_info already set */
        return OK;
     }
@@ -810,8 +810,7 @@ AP_CORE_DECLARE_NONSTD(apr_status_t) ap_sub_req_output_filter(ap_filter_t *f,
     if (APR_BUCKET_IS_EOS(e)) {
         apr_bucket_delete(e);
     }
-    ap_pass_brigade(f->next, bb);
-    return APR_SUCCESS;
+    return ap_pass_brigade(f->next, bb);
 }