Fix GH-23617: Keep a past-the-end internal pointer past the end on compaction - #23620

Hi @darkdi thanks for the patch but I m afraid the fix is really incorrect, I ll let ilaal review tough.

Direction looks right: matching the immutable path lines up with the MIN(nInternalPointer, nNumUsed) clamp tail deletion already uses, and an exhausted cursor picking up a later append is what the same code does with no separation at all. Two problems though.

The .phpt expectation is wrong and the test fails on this branch. end(); next(); then an append gives "d" at refcount 1 with no copy involved, so NULL is not a behaviour being restored. The test body prints "d", 4, 3, 4 here.

And it is incomplete for mixed arrays with holes, where zend_array_dup_elements() compacts and target->nNumUsed ends up below the preserved pointer:

Setting target->nInternalPointer = idx fixes row 2 but breaks row 3, since a full table rehashes on the append and zend_hash_rehash() does not remap an exhausted pointer either. So no single copied value agrees with the middle column everywhere; normalising the exhausted cursor wherever compaction happens, rehash included, looks like the actual fix.

The NULL expectation came from my issue text, which was wrong. I have corrected GH-23617.

Thanks, you were right on both. Reworked it the way you suggested: the exhausted cursor now goes to the new nNumUsed after compaction in both zend_array_dup_elements() and zend_hash_rehash(), not just copied. Built locally this time, your three rows plus packed with and without holes all give the appended element now, separated or not, and the new test fails on master for the holes cases. Zend/tests and ext/standard/tests/array pass