Fix #2774: shared hooks removal caused xdebug end handler to be skipped

[bwoebi]

Fixing #2774, just copy pasting the necessary logic from normal entry removal.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 78.33%. Comparing base (14de5d5) to head (e928685).

Files	Patch %	Lines
zend_abstract_interface/hook/hook.c	0.00%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2780      +/-   ##
============================================
- Coverage     80.81%   78.33%   -2.48%     
  Complexity     2452     2452              
============================================
  Files           144      170      +26     
  Lines         14329    18124    +3795     
  Branches          0      913     +913     
============================================
+ Hits          11580    14198    +2618     
- Misses         2749     3431     +682     
- Partials          0      495     +495     

Flag	Coverage Δ
appsec-extension	69.27% <ø> (?)
tracer-extension	78.19% <0.00%> (-0.30%)	⬇️
tracer-php	81.80% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
zend_abstract_interface/hook/hook.c	54.49% <0.00%> (-0.29%)	⬇️

... and 27 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 14de5d5...e928685. Read the comment docs.

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-08-20 13:01:37

Comparing candidate commit fe27b08 in PR branch bob/fix-2774 with baseline commit d75a84d in branch master.

Found 0 performance improvements and 1 performance regressions! Performance is the same for 177 metrics, 0 unstable metrics.

scenario:MessagePackSerializationBench/benchMessagePackSerialization

• 🟥 execution_time [+3.840µs; +5.540µs] or [+2.676%; +3.860%]

[iamluc]

Nice!

But some paths in the xdebug tests must be fixed