Thanks for the test case, that's much appreciated. Just a few comments:
- We might as well extend the existing trap-crasher test instead of adding a new one that is a near duplicate.
- The test should ideally be added in its own commit before the fix so that it's easy to verify that the fix actually fixes the test case
- We need all patches to go through the mailing list, including this updated one. I have added you to the mailman whitelist, so it won't get stuck in moderation if you send it.
Thanks for the test case, that's much appreciated. Just a few comments:
- We might as well extend the existing trap-crasher test instead of adding a new one that is a near duplicate.
- The test should ideally be added in its own commit before the fix so that it's easy to verify that the fix actually fixes the test case
- We need all patches to go through the mailing list, including this updated one. I have added you to the mailman whitelist, so it won't get stuck in moderation if you send it.