On 1/9/23 1:01 PM, jor...@apache.org wrote:
> Author: jorton
> Date: Mon Jan 9 12:01:56 2023
> New Revision: 1906487
>
> URL: http://svn.apache.org/viewvc?rev=1906487&view=rev
> Log:
> * modules/dav/main/util.c (dav_process_if_header): Fix error
> path for "Not" prefix parsing.
>
> Modified:
> httpd/httpd/trunk/modules/dav/main/util.c
>
> Modified: httpd/httpd/trunk/modules/dav/main/util.c
> URL:
> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/dav/main/util.c?rev=1906487&r1=1906486&r2=1906487&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/modules/dav/main/util.c (original)
> +++ httpd/httpd/trunk/modules/dav/main/util.c Mon Jan 9 12:01:56 2023
> @@ -801,8 +801,14 @@ static dav_error * dav_process_if_header
> "for the same state.");
> }
> condition = DAV_IF_COND_NOT;
> + list += 2;
> + }
> + else {
> + return dav_new_error(r->pool, HTTP_BAD_REQUEST,
> + DAV_ERR_IF_UNK_CHAR, 0,
> + "Invalid \"If:\" header: "
> + "Unexpected character in List");
Do we want to return here to save cycles or do we do another cycle in the while
loop and reuse the error message from 'default:'
and thus also ignore single 'N' s that are followed by a ' \t<['?
Regards
Rüdiger
