Skip to content

Commit bf036fa

Browse files
Zend/zend_globals: convert fiber_stack_size to size_t (#10619)
`zend_long` is a signed integer that's only 32 bit on some 64 bit architectures (e.g. ARM64). The proper type for memory sizes is `size_t`, and this type is accepted by zend_fiber_init_context().
1 parent d46dea1 commit bf036fa

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Zend/zend.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@ static ZEND_INI_MH(OnUpdateReservedStackSize) /* {{{ */
223223
static ZEND_INI_MH(OnUpdateFiberStackSize) /* {{{ */
224224
{
225225
if (new_value) {
226-
EG(fiber_stack_size) = zend_ini_parse_quantity_warn(new_value, entry->name);
226+
EG(fiber_stack_size) = zend_ini_parse_uquantity_warn(new_value, entry->name);
227227
} else {
228228
EG(fiber_stack_size) = ZEND_FIBER_DEFAULT_C_STACK_SIZE;
229229
}

Zend/zend_globals.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ struct _zend_executor_globals {
269269
zend_fiber *active_fiber;
270270

271271
/* Default fiber C stack size. */
272-
zend_long fiber_stack_size;
272+
size_t fiber_stack_size;
273273

274274
/* If record_errors is enabled, all emitted diagnostics will be recorded,
275275
* in addition to being processed as usual. */

0 commit comments

Comments
 (0)