Hi,

> The fix looks good to me. However, when I tried running the test with the fix 
> reverted, the test hung indefinitely, which doesn't seem desirable. Ideally, 
> the test should fail rather than hang when the fix is absent. It’s late here, 
> I haven’t spent time debugging why the test hang.

Thanks for testing. The hang is an expected consequence of the bug. The
resource counts are reset after release, but the hash slots remain
occupied. On reuse, the low counts can prevent a resize even when there
are no free slots, so ResourceOwnerAddToHash() keeps probing
indefinitely.

I have adjusted the test cases in v2 so that they still expose the bug
without hanging when the fix is absent.

I verified that the revised regression passes with the fix and fails
promptly with duplicate-release errors when only the fix is reverted.

Attached is v2.

---
Regards,
Jingtang

Attachment: v2-aux-resource-owner-reuse.patch
Description: Binary data

Reply via email to