You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by gb...@apache.org on 2023/06/08 09:24:21 UTC
svn commit: r1910304 - /httpd/httpd/trunk/modules/filters/mod_ext_filter.c
Author: gbechis
Date: Thu Jun 8 09:24:21 2023
New Revision: 1910304
URL: http://svn.apache.org/viewvc?rev=1910304&view=rev
Log:
fix return codes
spotted by Yann Ylavic
Modified:
httpd/httpd/trunk/modules/filters/mod_ext_filter.c
Modified: httpd/httpd/trunk/modules/filters/mod_ext_filter.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/filters/mod_ext_filter.c?rev=1910304&r1=1910303&r2=1910304&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/filters/mod_ext_filter.c (original)
+++ httpd/httpd/trunk/modules/filters/mod_ext_filter.c Thu Jun 8 09:24:21 2023
@@ -747,13 +747,13 @@ static apr_status_t check_filter_process
ap_log_rerror(APLOG_MARK, APLOG_ERR, APR_SUCCESS, r, APLOGNO(10452)
"child process %s killed by signal %d, uri=%s",
ctx->filter->command, exitcode, r->uri);
- return HTTP_INTERNAL_SERVER_ERROR;
+ return APR_EGENERAL;
}
else if (exitcode != 0) {
ap_log_rerror(APLOG_MARK, APLOG_ERR, APR_SUCCESS, r, APLOGNO(10453)
"child process %s exited with non-zero status %d, "
"uri=%s", ctx->filter->command, exitcode, r->uri);
- return HTTP_INTERNAL_SERVER_ERROR;
+ return APR_EGENERAL;
}
}
@@ -766,7 +766,7 @@ static apr_status_t check_filter_process
* bb, dropping the previous content of bb (the input)
*/
-static int ef_unified_filter(ap_filter_t *f, apr_bucket_brigade *bb)
+static apr_status_t ef_unified_filter(ap_filter_t *f, apr_bucket_brigade *bb)
{
request_rec *r = f->r;
conn_rec *c = r->connection;