Skip to content

Windows: Handles - catch exception in handle iteration#1598

Merged
ikelos merged 1 commit intodevelopfrom
issue_1573_handle_iteration_traceback
Feb 3, 2025
Merged

Windows: Handles - catch exception in handle iteration#1598
ikelos merged 1 commit intodevelopfrom
issue_1573_handle_iteration_traceback

Conversation

@dgmcdona
Copy link
Contributor

An InvalidAddressException can occur inside of __iter__ when
iterating over the handle table (the exact exception occurs when
creating the subtype in objects.Array.__getitem__. This changes the
handle code to do a manual iteration over the sequence using the array
length and indexes, catch the exception, log the index, and continue.

In the test sample that prompted this change, the exception occurred on
the access of the very last item in the array.

closes #1573

@dgmcdona dgmcdona force-pushed the issue_1573_handle_iteration_traceback branch from 718d439 to 38501f7 Compare January 31, 2025 23:34
An `InvalidAddressException` can occur inside of `__iter__` when
iterating over the handle table (the exact exception occurs when
creating the subtype in `objects.Array.__getitem__`. This changes the
handle code to do a manual iteration over the sequence using the array
length and indexes, catch the exception, log the index, and continue.

In the test sample that prompted this change, the exception occurred on
the access of the very last item in the array.

closes #1573
@dgmcdona dgmcdona force-pushed the issue_1573_handle_iteration_traceback branch from 38501f7 to 1d1af69 Compare January 31, 2025 23:35
@ikelos ikelos added the awaiting-author-response This issue/pull request needs attention from the original author label Feb 1, 2025
@ikelos ikelos merged commit a4987b3 into develop Feb 3, 2025
24 checks passed
@ikelos ikelos deleted the issue_1573_handle_iteration_traceback branch February 3, 2025 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-author-response This issue/pull request needs attention from the original author parity-release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle bug round 2

4 participants