Stack overflow with nested brackets
Rafael EspĂndola
rafael.espindola at gmail.com
Wed Jul 24 19:13:53 PDT 2013
On 24 July 2013 15:34, Aaron Ballman <aaron at aaronballman.com> wrote:
> Patch LGTM; is there a way that we could add a test to ensure
> diagnoseOverflow doesn't break again (since this is fixing a
> regression)?
Thanks, I committed it as r187097.
I found this by enabling pipefail and running the tests on a windows
vm. I hope to be able to enable it really soon now. Once that is in,
parser_overflow.c should fail on any systems that goes out of stack
space. I guess we could explicitly run ulimit in the test, but I am
not sure that adds a lot of value.
> ~Aaron
Cheers,
Rafael
More information about the cfe-commits
mailing list