diff options
author | Father Chrysostomos <sprout@cpan.org> | 2011-06-24 08:17:28 -0700 |
---|---|---|
committer | Father Chrysostomos <sprout@cpan.org> | 2011-06-24 08:19:28 -0700 |
commit | 6d1c68e64c4dbbb1c7e8a28de678cd3247649bad (patch) | |
tree | 429f168000f8db2ec45dd6e3ff555fb90e465f08 /form.h | |
parent | f4e68e82b126f2b2bfde4c660b1045c1e4a69455 (diff) | |
download | perl-6d1c68e64c4dbbb1c7e8a28de678cd3247649bad.tar.gz |
[perl #93454] Free deleted iterator when freeing hash
Commit 7d6175e, which did a fix-up after commit e0171a1a3, which
introduced hfree_next_entry, did not account for the fact that
hfree_next_entry frees the hash iterator before removing and returning
the next value.
It changed the callers to check the number of keys to determine
whether anything else needed to be freed, which meant that
hfree_next_entry was called one time less than necessary on hashes
whose current iterator had been deleted and which consequently
appeared empty.
This fixes that.
I don’t know how to test it, but the string table warnings were caus-
ing test failures on VMS, so maybe that’s good enough.
Diffstat (limited to 'form.h')
0 files changed, 0 insertions, 0 deletions