Skip to content

Stdlib: socket_addrinfo_lookup(null) soft-null Deprecated+false (#30337) - #30366

Merged
PurHur merged 1 commit into
masterfrom
fix/30337-socket-addrinfo-lookup-soft-null
Aug 11, 2026
Merged

Stdlib: socket_addrinfo_lookup(null) soft-null Deprecated+false (#30337)#30366
PurHur merged 1 commit into
masterfrom
fix/30337-socket-addrinfo-lookup-soft-null

Conversation

@PurHur

@PurHur PurHur commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #30337

php-src reference

  • ext/sockets/sockets.cPHP_FUNCTION(socket_addrinfo_lookup) host Z_PARAM_STR
  • Implementation: ext/sockets/socket_addrinfo_lookup.php

Test plan

  • Issue repro Zend vs VM
  • vendor/bin/phpunit --filter SocketAddrinfoLookupSoftNullVMTest (2 tests OK)

Verification

=== VM soft ===
PHP Deprecated:  socket_addrinfo_lookup(): Passing null to parameter #1 ($host) of type string is deprecated …
false
=== VM strict ===
TypeError: socket_addrinfo_lookup(): Argument #1 ($host) must be of type string, null given
OK (2 tests, 2 assertions)

Not covered: full ci-fast / JIT (builtin is VM-only for addrinfo_lookup).

Made with Cursor

Match Z_PARAM_STR soft-forward outside strict_types; keep TypeError under strict_types.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit cb6feb0 into master Aug 11, 2026
1 check failed
@PurHur
PurHur deleted the fix/30337-socket-addrinfo-lookup-soft-null branch August 11, 2026 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Regression: socket_addrinfo_lookup(null) soft TypeError — Zend Deprecated+false (ext/sockets/sockets.c)

1 participant