Skip to content

Commit 691fa14

Browse files
committed
Revert to original check with long parameter to unsigned int conversion
1 parent dab9d20 commit 691fa14

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

ext/sockets/sockets.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2331,16 +2331,17 @@ PHP_FUNCTION(socket_set_option)
23312331

23322332
#if defined(TCP_USER_TIMEOUT)
23332333
case TCP_USER_TIMEOUT: {
2334-
ov = zval_get_long(arg4);
2334+
zend_long timeout timeout = zval_get_long(arg4);
23352335

23362336
// TCP_USER_TIMEOUT unsigned int
2337-
if (ov < 0 || ov > UINT_MAX) {
2337+
if (timeout < 0 || timeout > UINT_MAX) {
23382338
zend_argument_value_error(4, "must be of between 0 and %u", UINT_MAX);
23392339
RETURN_THROWS();
23402340
}
23412341

2342-
optlen = sizeof(ov);
2343-
opt_ptr = &ov;
2342+
unsigned int val = (unsigned int)timeout;
2343+
optlen = sizeof(val);
2344+
opt_ptr = &val;
23442345
break;
23452346
}
23462347
#endif

0 commit comments

Comments
 (0)