[PATCH] Fix cygcheck -p's handling of '+'
Corinna Vinschen
corinna-cygwin@cygwin.com
Tue Jan 31 08:48:00 GMT 2017
On Jan 30 15:37, Jon Turney wrote:
> The form data sent to the server should be application/x-www-form-urlencoded
>
> This replaces spaces with '+' before being RFC 11738 encoded, so a literal
> '+' must be %-encoded also.
>
> See https://cygwin.com/ml/cygwin/2014-01/msg00287.html et seq.
>
> Signed-off-by: Jon Turney <jon.turney@dronecode.org.uk>
> ---
> winsup/utils/cygcheck.cc | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/winsup/utils/cygcheck.cc b/winsup/utils/cygcheck.cc
> index d1e27b7..e745b20 100644
> --- a/winsup/utils/cygcheck.cc
> +++ b/winsup/utils/cygcheck.cc
> @@ -2009,8 +2009,8 @@ check_keys ()
> return 0;
> }
>
> -/* RFC1738 says that these do not need to be escaped. */
> -static const char safe_chars[] = "$-_.+!*'(),";
> +/* These do not need to be escaped in application/x-www-form-urlencoded */
> +static const char safe_chars[] = "$-_.!*'(),";
>
> /* the URL to query. */
> static const char base_url[] =
> --
> 2.8.3
Thanks, please apply.
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Maintainer cygwin AT cygwin DOT com
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20170131/706870c5/attachment.sig>
More information about the Cygwin-patches
mailing list