GeezerVison

Destructor functions stored in hash/llist dtor slots were defined with
typed parameters and cast to the generic void(*)(void*) slot type. UBSan's
-fsanitize=function flags calls through a mismatched pointer as undefined
behavior, aborting in post-deactivate during request shutdown and breaking
dbgp tests downstream.

Change the dtors to take void* and cast internally so the pointer type
matches the callee's real signature; drop the now-unnecessary casts at
registration sites. Wrap zend_string_release in a local dtor since its
signature is fixed.

…signature

carlos-granados left a comment

…-type check"

This reverts commit 7dfa442 and the follow-up 6dbefbb. The function-type
cast pattern is preserved as-is in upstream Xdebug; rewriting dtors to
take void* would create ongoing merge friction for a benign-in-practice
UBSan diagnostic. Suppressing the check at the CI level instead.

The dtor registration pattern (casting typed destructors to the generic
xdebug hash/llist dtor slot types) is shared with upstream Xdebug and
works correctly on all supported platforms, but trips UBSan's
-fsanitize=function indirect-call type check. Suppress the check at
build time rather than diverge from upstream source.

carlos-granados left a comment