X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-http;a=blobdiff_plain;f=http_curl_api.c;h=7938573eb6bfa962f50cf1bd10f59498a5bcbe30;hp=e94e5dc732d5446e87022f22f4fc1ff2c7491541;hb=2b7331c5fdb73cb48f7f60a8e4ec88766581ced3;hpb=8cdf77bbae1974a94d69ab757cbda38b88299e5f

diff --git a/http_curl_api.c b/http_curl_api.c
index e94e5dc..7938573 100644
--- a/http_curl_api.c
+++ b/http_curl_api.c
@@ -85,8 +85,9 @@ static inline char *_http_curl_copystr(const char *str TSRMLS_DC);
 #define http_curl_setopts(c, u, o, r) _http_curl_setopts((c), (u), (o), (r) TSRMLS_CC)
 static void _http_curl_setopts(CURL *ch, const char *url, HashTable *options, phpstr *response TSRMLS_DC);
 
-#define http_curl_getopt(o, k, t) _http_curl_getopt((o), (k), (t) TSRMLS_CC)
-static inline zval *_http_curl_getopt(HashTable *options, char *key, int type TSRMLS_DC);
+#define http_curl_getopt(o, k, t) _http_curl_getopt_ex((o), (k), sizeof(k), (t) TSRMLS_CC)
+#define http_curl_getopt_ex(o, k, l, t) _http_curl_getopt_ex((o), (k), (l), (t) TSRMLS_CC)
+static inline zval *_http_curl_getopt_ex(HashTable *options, char *key, size_t keylen, int type TSRMLS_DC);
 
 static size_t http_curl_callback(char *, size_t, size_t, void *);
 
@@ -105,17 +106,16 @@ static inline char *_http_curl_copystr(const char *str TSRMLS_DC)
 /* {{{ static size_t http_curl_callback(char *, size_t, size_t, void *) */
 static size_t http_curl_callback(char *buf, size_t len, size_t n, void *s)
 {
-	phpstr_append(PHPSTR(s), buf, len *= n);
-	return len;
+	return phpstr_append(PHPSTR(s), buf, len *= n);
 }
 /* }}} */
 
-/* {{{ static inline zval *http_curl_getopt(HashTable *, char *, int) */
-static inline zval *_http_curl_getopt(HashTable *options, char *key, int type TSRMLS_DC)
+/* {{{ static inline zval *http_curl_getopt(HashTable *, char *, size_t, int) */
+static inline zval *_http_curl_getopt_ex(HashTable *options, char *key, size_t keylen, int type TSRMLS_DC)
 {
 	zval **zoption;
 
-	if (SUCCESS != zend_hash_find(options, key, strlen(key) + 1, (void **) &zoption)) {
+	if (!options || (SUCCESS != zend_hash_find(options, key, keylen, (void **) &zoption))) {
 		return NULL;
 	}
 
@@ -146,15 +146,22 @@ static void _http_curl_setopts(CURL *ch, const char *url, HashTable *options, ph
 #define HTTP_CURL_OPT(OPTION, p) curl_easy_setopt(ch, CURLOPT_##OPTION, (p))
 
 	/* standard options */
-	HTTP_CURL_OPT(URL, url);
+	if (url) {
+		HTTP_CURL_OPT(URL, url);
+	}
+
 	HTTP_CURL_OPT(HEADER, 0);
 	HTTP_CURL_OPT(FILETIME, 1);
 	HTTP_CURL_OPT(NOPROGRESS, 1);
 	HTTP_CURL_OPT(AUTOREFERER, 1);
 	HTTP_CURL_OPT(WRITEFUNCTION, http_curl_callback);
 	HTTP_CURL_OPT(HEADERFUNCTION, http_curl_callback);
-	HTTP_CURL_OPT(WRITEDATA, response);
-	HTTP_CURL_OPT(WRITEHEADER, response);
+
+	if (response) {
+		HTTP_CURL_OPT(WRITEDATA, response);
+		HTTP_CURL_OPT(WRITEHEADER, response);
+	}
+
 #if defined(ZTS) && (LIBCURL_VERSION_NUM >= 0x070a00)
 	HTTP_CURL_OPT(NOSIGNAL, 1);
 #endif