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