From 656cce68348695162120584f5c12ceeed16b2792 Mon Sep 17 00:00:00 2001 From: Fabian Keil Date: Thu, 31 Aug 2006 10:55:49 +0000 Subject: [PATCH] Block requests for untrusted URLs with status code 403 instead of 200. --- filters.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/filters.c b/filters.c index 7eb67334..1a1da409 100644 --- a/filters.c +++ b/filters.c @@ -1,4 +1,4 @@ -const char filters_rcs[] = "$Id: filters.c,v 1.62 2006/08/14 00:27:47 david__schmidt Exp $"; +const char filters_rcs[] = "$Id: filters.c,v 1.63 2006/08/31 10:11:28 fabiankeil Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/filters.c,v $ @@ -39,6 +39,11 @@ const char filters_rcs[] = "$Id: filters.c,v 1.62 2006/08/14 00:27:47 david__sch * * Revisions : * $Log: filters.c,v $ + * Revision 1.63 2006/08/31 10:11:28 fabiankeil + * Don't free p which is still in use and will be later + * freed by free_map(). Don't claim the referrer is unknown + * when the client didn't set one. + * * Revision 1.62 2006/08/14 00:27:47 david__schmidt * Feature request 595948: Re-Filter logging in single line * @@ -968,7 +973,7 @@ struct http_response *block_url(struct client_state *csp) * Function : trust_url FIXME: I should be called distrust_url * * Description : Calls is_untrusted_url to determine if the URL is trusted - * and if not, returns a HTTP 304 response with a reject message. + * and if not, returns a HTTP 403 response with a reject message. * * Parameters : * 1 : csp = Current client state (buffers, headers, etc...) @@ -1002,8 +1007,9 @@ struct http_response *trust_url(struct client_state *csp) return cgi_error_memory(); } + rsp->status = strdup("403 Request blocked by Privoxy"); exports = default_exports(csp, NULL); - if (exports == NULL) + if (exports == NULL || rsp->status == NULL) { free_http_response(rsp); return cgi_error_memory(); -- 2.49.0