From: Michael Wallner <mike@php.net>
Date: Tue, 4 Apr 2017 11:35:18 +0000 (+0200)
Subject: Fixed issue #63
X-Git-Tag: RELEASE_3_2_0_RC1~52
X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=commitdiff_plain;h=aff646843d13892c5aa3ada2de08e0ab1f2f6ad8;hp=ba994a971deb5cd596f2348e865117bd4208a093

Fixed issue #63

Use urldecode instead of raw_urldecode in http_params
---

diff --git a/src/php_http_params.c b/src/php_http_params.c
index 9f27918..9b40cfe 100644
--- a/src/php_http_params.c
+++ b/src/php_http_params.c
@@ -123,7 +123,7 @@ static inline void prepare_escaped(zval *zv)
 
 static inline void sanitize_urlencoded(zval *zv)
 {
-	Z_STRLEN_P(zv) = php_raw_url_decode(Z_STRVAL_P(zv), Z_STRLEN_P(zv));
+	Z_STRLEN_P(zv) = php_url_decode(Z_STRVAL_P(zv), Z_STRLEN_P(zv));
 }
 
 static inline void prepare_urlencoded(zval *zv)
diff --git a/tests/gh-issue63.phpt b/tests/gh-issue63.phpt
new file mode 100644
index 0000000..e03b32f
--- /dev/null
+++ b/tests/gh-issue63.phpt
@@ -0,0 +1,27 @@
+--TEST--
+segfault with Client::setDebug and Client::dequeue()
+--SKIPIF--
+<?php
+include "skipif.inc";
+skip_client_test();
+skip_online_test();
+?>
+--FILE--
+<?php
+echo "Test\n";
+
+use http\QueryString;
+use http\Url;
+
+$url = 'http://example.com/?param=has+spaces';
+$query = ['foo' => 'bar'];
+$parts['query'] = new QueryString($query);
+echo (new Url($url, null, Url::STDFLAGS))->mod($parts)->toString();
+
+?>
+
+===DONE===
+--EXPECTF--
+Test
+http://example.com/?param=has%20spaces&foo=bar
+===DONE===