Skip to content

Commit c2030b3

Browse files
kocsismatekocsismate
committed
Address code review
1 parent e4de3b4 commit c2030b3

15 files changed

+96
-87
lines changed

‎ext/curl/curl.stub.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ function curl_multi_errno(CurlMulti $multi_handle): int {}
3737

3838
function curl_multi_exec(CurlMulti $multi_handle, &$still_running): int {}
3939

40-
function curl_multi_getcontent(CurlMulti $multi_handle): ?string {}
40+
function curl_multi_getcontent(Curl $multi_handle): ?string {}
4141

4242
function curl_multi_info_read(CurlMulti $multi_handle, &$msgs_in_queue = null): array|false {}
4343

‎ext/curl/curl_arginfo.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_curl_multi_exec, 0, 2, IS_LONG,
7373
ZEND_END_ARG_INFO()
7474

7575
ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(arginfo_curl_multi_getcontent, 0, 1, IS_STRING, 1)
76-
ZEND_ARG_OBJ_INFO(0, multi_handle, CurlMulti, 0)
76+
ZEND_ARG_OBJ_INFO(0, multi_handle, Curl, 0)
7777
ZEND_END_ARG_INFO()
7878

7979
ZEND_BEGIN_ARG_WITH_RETURN_TYPE_MASK_EX(arginfo_curl_multi_info_read, 0, 1, MAY_BE_ARRAY|MAY_BE_FALSE)

‎ext/curl/interface.c

Lines changed: 27 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
#endif
2222

2323
#include "php.h"
24+
#include "Zend/zend_interfaces.h"
2425

2526
#if HAVE_CURL
2627

@@ -231,6 +232,7 @@ ZEND_GET_MODULE (curl)
231232
/* Curl class */
232233

233234
zend_class_entry *curl_ce;
235+
zend_class_entry *curl_share_ce;
234236
static zend_object_handlers curl_object_handlers;
235237

236238
static zend_object *curl_create_object(zend_class_entry *class_type);
@@ -1190,15 +1192,18 @@ PHP_MINIT_FUNCTION(curl)
11901192

11911193
zend_class_entry ce;
11921194
INIT_CLASS_ENTRY(ce, "Curl", curl_object_methods);
1193-
ce.create_object = curl_create_object;
1194-
ce.ce_flags |= ZEND_ACC_FINAL;
11951195
curl_ce = zend_register_internal_class(&ce);
1196+
curl_ce->ce_flags |= ZEND_ACC_FINAL;
1197+
curl_ce->create_object = curl_create_object;
1198+
curl_ce->serialize = zend_class_serialize_deny;
1199+
curl_ce->unserialize = zend_class_unserialize_deny;
11961200

11971201
memcpy(&curl_object_handlers, &std_object_handlers, sizeof(zend_object_handlers));
11981202
curl_object_handlers.offset = XtOffsetOf(php_curl, std);
11991203
curl_object_handlers.free_obj = curl_free_obj;
12001204
curl_object_handlers.get_gc = curl_get_gc;
12011205
curl_object_handlers.get_constructor = curl_get_constructor;
1206+
curl_object_handlers.clone_obj = NULL;
12021207

12031208
curl_multi_register_class();
12041209
curl_share_register_class();
@@ -1212,7 +1217,6 @@ PHP_MINIT_FUNCTION(curl)
12121217

12131218
static zend_object *curl_create_object(zend_class_entry *class_type) {
12141219
php_curl *intern = zend_object_alloc(sizeof(php_curl), class_type);
1215-
memset(intern, 0, sizeof(php_curl) - sizeof(zend_object));
12161220

12171221
zend_object_std_init(&intern->std, class_type);
12181222
object_properties_init(&intern->std, class_type);
@@ -1300,8 +1304,8 @@ static size_t curl_write(char *data, size_t size, size_t nmemb, void *ctx)
13001304
int error;
13011305
zend_fcall_info fci;
13021306

1303-
GC_ADDREF(ch->res);
1304-
ZVAL_RES(&argv[0], ch->res);
1307+
GC_ADDREF(&ch->std);
1308+
ZVAL_OBJ(&argv[0], &ch->std);
13051309
ZVAL_STRINGL(&argv[1], data, length);
13061310

13071311
fci.size = sizeof(fci);
@@ -1347,8 +1351,8 @@ static int curl_fnmatch(void *ctx, const char *pattern, const char *string)
13471351
int error;
13481352
zend_fcall_info fci;
13491353

1350-
GC_ADDREF(ch->res);
1351-
ZVAL_RES(&argv[0], ch->res);
1354+
GC_ADDREF(&ch->std);
1355+
ZVAL_OBJ(&argv[0], &ch->std);
13521356
ZVAL_STRING(&argv[1], pattern);
13531357
ZVAL_STRING(&argv[2], string);
13541358

@@ -1399,8 +1403,8 @@ static size_t curl_progress(void *clientp, double dltotal, double dlnow, double
13991403
int error;
14001404
zend_fcall_info fci;
14011405

1402-
GC_ADDREF(ch->res);
1403-
ZVAL_RES(&argv[0], ch->res);
1406+
GC_ADDREF(&ch->std);
1407+
ZVAL_OBJ(&argv[0], &ch->std);
14041408
ZVAL_LONG(&argv[1], (zend_long)dltotal);
14051409
ZVAL_LONG(&argv[2], (zend_long)dlnow);
14061410
ZVAL_LONG(&argv[3], (zend_long)ultotal);
@@ -1453,8 +1457,8 @@ static size_t curl_read(char *data, size_t size, size_t nmemb, void *ctx)
14531457
int error;
14541458
zend_fcall_info fci;
14551459

1456-
GC_ADDREF(ch->res);
1457-
ZVAL_RES(&argv[0], ch->res);
1460+
GC_ADDREF(&ch->std);
1461+
ZVAL_OBJ(&argv[0], &ch->std);
14581462
if (t->res) {
14591463
GC_ADDREF(t->res);
14601464
ZVAL_RES(&argv[1], t->res);
@@ -1522,8 +1526,8 @@ static size_t curl_write_header(char *data, size_t size, size_t nmemb, void *ctx
15221526
int error;
15231527
zend_fcall_info fci;
15241528

1525-
GC_ADDREF(ch->res);
1526-
ZVAL_RES(&argv[0], ch->res);
1529+
GC_ADDREF(&ch->std);
1530+
ZVAL_OBJ(&argv[0], &ch->std);
15271531
ZVAL_STRINGL(&argv[1], data, length);
15281532

15291533
fci.size = sizeof(fci);
@@ -1715,9 +1719,8 @@ php_curl *alloc_curl_handle(zval *curl)
17151719

17161720
ch->to_free->slist = emalloc(sizeof(HashTable));
17171721
zend_hash_init(ch->to_free->slist, 4, NULL, curl_free_slist, 0);
1718-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
17191722
ZVAL_UNDEF(&ch->postfields);
1720-
#endif
1723+
17211724
return ch;
17221725
}
17231726
/* }}} */
@@ -1822,7 +1825,7 @@ PHP_FUNCTION(curl_init)
18221825

18231826
if (url) {
18241827
if (php_curl_option_url(ch, ZSTR_VAL(url), ZSTR_LEN(url)) == FAILURE) {
1825-
curl_free_obj(Z_OBJ_P(return_value));
1828+
zval_ptr_dtor(return_value);
18261829
RETURN_FALSE;
18271830
}
18281831
}
@@ -2015,8 +2018,8 @@ static inline int build_mime_structure_from_hash(php_curl *ch, zval *zpostfields
20152018
filename = Z_STRVAL_P(prop);
20162019
}
20172020

2018-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
20192021
zval_ptr_dtor(&ch->postfields);
2022+
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
20202023
ZVAL_COPY(&ch->postfields, zpostfields);
20212024

20222025
if ((stream = php_stream_open_wrapper(ZSTR_VAL(postval), "rb", STREAM_MUST_SEEK, NULL))) {
@@ -2124,9 +2127,7 @@ PHP_FUNCTION(curl_copy_handle)
21242127
CURL *cp;
21252128
zval *zid;
21262129
php_curl *dupch;
2127-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
21282130
zval *postfields;
2129-
#endif
21302131

21312132
ZEND_PARSE_PARAMETERS_START(1,1)
21322133
Z_PARAM_OBJECT_OF_CLASS(zid, curl_ce)
@@ -2145,16 +2146,14 @@ PHP_FUNCTION(curl_copy_handle)
21452146

21462147
_php_setup_easy_copy_handlers(dupch, ch);
21472148

2148-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
21492149
postfields = &ch->postfields;
21502150
if (Z_TYPE_P(postfields) != IS_UNDEF) {
21512151
if (build_mime_structure_from_hash(dupch, postfields) != SUCCESS) {
2152-
curl_free_obj(Z_OBJ_P(return_value));
2152+
zval_ptr_dtor(return_value);
21532153
php_error_docref(NULL, E_WARNING, "Cannot rebuild mime structure");
21542154
RETURN_FALSE;
21552155
}
21562156
}
2157-
#endif
21582157
}
21592158
/* }}} */
21602159

@@ -2784,8 +2783,12 @@ static int _php_curl_setopt(php_curl *ch, zend_long option, zval *zvalue) /* {{{
27842783

27852784
case CURLOPT_SHARE:
27862785
{
2787-
php_curlsh *sh = Z_CURL_SHARE_P(zvalue);
2788-
curl_easy_setopt(ch->cp, CURLOPT_SHARE, sh->share);
2786+
if (Z_TYPE_P(zvalue) == IS_OBJECT && Z_OBJCE_P(zvalue) == curl_share_ce) {
2787+
php_curlsh *sh = Z_CURL_SHARE_P(zvalue);
2788+
curl_easy_setopt(ch->cp, CURLOPT_SHARE, sh->share);
2789+
} else {
2790+
return FAILURE;
2791+
}
27892792
}
27902793
break;
27912794

@@ -3323,13 +3326,9 @@ static void curl_free_obj(zend_object *object)
33233326
}
33243327

33253328
efree(ch->handlers);
3326-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
33273329
zval_ptr_dtor(&ch->postfields);
3328-
#endif
33293330

33303331
zend_object_std_dtor(&ch->std);
3331-
3332-
efree(ch);
33333332
}
33343333
/* }}} */
33353334

‎ext/curl/multi.c

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
#endif
2222

2323
#include "php.h"
24+
#include "Zend/zend_interfaces.h"
2425

2526
#if HAVE_CURL
2627

@@ -49,6 +50,8 @@
4950

5051
/* CurlMulti class */
5152

53+
static zend_class_entry *curl_multi_ce;
54+
5255
static inline php_curlm *curl_multi_from_obj(zend_object *obj) {
5356
return (php_curlm *)((char *)(obj) - XtOffsetOf(php_curlm, std));
5457
}
@@ -229,7 +232,7 @@ PHP_FUNCTION(curl_multi_exec)
229232

230233
for (pz_ch = (zval *)zend_llist_get_first_ex(&mh->easyh, &pos); pz_ch;
231234
pz_ch = (zval *)zend_llist_get_next_ex(&mh->easyh, &pos)) {
232-
ch = Z_CURL_P(pz_ch);
235+
ch = Z_CURL_P(pz_ch);
233236

234237
_php_curl_verify_handlers(ch, 1);
235238
}
@@ -251,7 +254,7 @@ PHP_FUNCTION(curl_multi_getcontent)
251254
zval *z_ch;
252255
php_curl *ch;
253256

254-
ZEND_PARSE_PARAMETERS_START(1,1)
257+
ZEND_PARSE_PARAMETERS_START(1, 1)
255258
Z_PARAM_OBJECT_OF_CLASS(z_ch, curl_ce)
256259
ZEND_PARSE_PARAMETERS_END();
257260

@@ -538,7 +541,6 @@ static const zend_function_entry curl_multi_methods[] = {
538541

539542
static zend_object *curl_multi_create_object(zend_class_entry *class_type) {
540543
php_curlm *intern = zend_object_alloc(sizeof(php_curlm), class_type);
541-
memset(intern, 0, sizeof(php_curlm) - sizeof(zend_object));
542544

543545
zend_object_std_init(&intern->std, class_type);
544546
object_properties_init(&intern->std, class_type);
@@ -581,22 +583,23 @@ void curl_multi_free_obj(zend_object *object)
581583
}
582584

583585
zend_object_std_dtor(&mh->std);
584-
585-
efree(mh);
586586
}
587587
}
588588

589589
void curl_multi_register_class(void) {
590590
zend_class_entry ce_multi;
591591
INIT_CLASS_ENTRY(ce_multi, "CurlMulti", curl_multi_methods);
592-
ce_multi.create_object = curl_multi_create_object;
593-
ce_multi.ce_flags |= ZEND_ACC_FINAL;
594592
curl_multi_ce = zend_register_internal_class(&ce_multi);
593+
curl_multi_ce->ce_flags |= ZEND_ACC_FINAL;
594+
curl_multi_ce->create_object = curl_multi_create_object;
595+
curl_multi_ce->serialize = zend_class_serialize_deny;
596+
curl_multi_ce->unserialize = zend_class_unserialize_deny;
595597

596598
memcpy(&curl_multi_handlers, &std_object_handlers, sizeof(zend_object_handlers));
597599
curl_multi_handlers.offset = XtOffsetOf(php_curlm, std);
598600
curl_multi_handlers.free_obj = curl_multi_free_obj;
599601
curl_multi_handlers.get_constructor = curl_multi_get_constructor;
602+
curl_multi_handlers.clone_obj = NULL;
600603
}
601604

602605
#endif

‎ext/curl/php_curl.h

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -117,16 +117,13 @@ struct _php_curl_free {
117117
typedef struct {
118118
CURL *cp;
119119
php_curl_handlers *handlers;
120-
zend_resource *res;
121120
struct _php_curl_free *to_free;
122121
struct _php_curl_send_headers header;
123122
struct _php_curl_error err;
124123
zend_bool in_callback;
125124
uint32_t* clone;
126-
#if LIBCURL_VERSION_NUM >= 0x073800 /* 7.56.0 */
127125
zval postfields;
128126
zend_object std;
129-
#endif
130127
} php_curl;
131128

132129
#define CURLOPT_SAFE_UPLOAD -1
@@ -173,14 +170,12 @@ static inline php_curlsh *curl_share_from_obj(zend_object *obj) {
173170
#define Z_CURL_SHARE_P(zv) curl_share_from_obj(Z_OBJ_P(zv))
174171

175172
PHP_CURL_API extern zend_class_entry *curl_ce;
173+
PHP_CURL_API extern zend_class_entry *curl_share_ce;
176174

177-
PHP_CURL_API extern zend_class_entry *curl_multi_ce;
178175
void curl_multi_register_class(void);
179-
180-
PHP_CURL_API extern zend_class_entry *curl_share_ce;
181176
void curl_share_register_class(void);
182-
183177
void curlfile_register_class(void);
178+
184179
PHP_CURL_API extern zend_class_entry *curl_CURLFile_class;
185180

186181
#else

‎ext/curl/share.c

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
#endif
2222

2323
#include "php.h"
24+
#include "Zend/zend_interfaces.h"
2425

2526
#if HAVE_CURL
2627

@@ -30,6 +31,8 @@
3031

3132
#define SAVE_CURLSH_ERROR(__handle, __err) (__handle)->err.no = (int) __err;
3233

34+
zend_class_entry *curl_share_ce;
35+
3336
/* {{{ proto CurlShare curl_share_init()
3437
Initialize a share curl handle */
3538
PHP_FUNCTION(curl_share_init)
@@ -151,7 +154,6 @@ static const zend_function_entry curl_share_methods[] = {
151154

152155
static zend_object *curl_share_create_object(zend_class_entry *class_type) {
153156
php_curlsh *intern = zend_object_alloc(sizeof(php_curlsh), class_type);
154-
memset(intern, 0, sizeof(php_curlsh) - sizeof(zend_object));
155157

156158
zend_object_std_init(&intern->std, class_type);
157159
object_properties_init(&intern->std, class_type);
@@ -169,24 +171,24 @@ void curl_share_free_obj(zend_object *object)
169171
{
170172
php_curlsh *sh = curl_share_from_obj(object);
171173

172-
if (sh) {
173-
curl_share_cleanup(sh->share);
174-
zend_object_std_dtor(&sh->std);
175-
efree(sh);
176-
}
174+
curl_share_cleanup(sh->share);
175+
zend_object_std_dtor(&sh->std);
177176
}
178177

179178
void curl_share_register_class(void) {
180179
zend_class_entry ce_share;
181180
INIT_CLASS_ENTRY(ce_share, "CurlShare", curl_share_methods);
182-
ce_share.create_object = curl_share_create_object;
183-
ce_share.ce_flags |= ZEND_ACC_FINAL;
184181
curl_share_ce = zend_register_internal_class(&ce_share);
182+
curl_share_ce->ce_flags |= ZEND_ACC_FINAL;
183+
curl_share_ce->create_object = curl_share_create_object;
184+
curl_share_ce->serialize = &zend_class_serialize_deny;
185+
curl_share_ce->unserialize = &zend_class_unserialize_deny;
185186

186187
memcpy(&curl_share_handlers, &std_object_handlers, sizeof(zend_object_handlers));
187188
curl_share_handlers.offset = XtOffsetOf(php_curlsh, std);
188189
curl_share_handlers.free_obj = curl_share_free_obj;
189190
curl_share_handlers.get_constructor = curl_share_get_constructor;
191+
curl_share_handlers.clone_obj = NULL;
190192
}
191193

192194
#endif

‎ext/curl/tests/bug48514.phpt

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,17 @@ if (!extension_loaded('curl')) {
1313

1414
$ch1 = curl_init();
1515
var_dump($ch1);
16-
var_dump(get_resource_type($ch1));
16+
var_dump($ch1::class);
1717

1818
$ch2 = curl_multi_init();
1919
var_dump($ch2);
20-
var_dump(get_resource_type($ch2));
20+
var_dump($ch2::class);
2121

2222
?>
23-
--EXPECTF--
24-
resource(%d) of type (curl)
25-
string(4) "curl"
26-
resource(%d) of type (curl_multi)
27-
string(10) "curl_multi"
23+
--EXPECT--
24+
object(Curl)#1 (0) {
25+
}
26+
string(4) "Curl"
27+
object(CurlMulti)#2 (0) {
28+
}
29+
string(9) "CurlMulti"

0 commit comments

Comments
 (0)