DOMjudge / domjudge

DOMjudge programming contest jury system
https://www.domjudge.org
GNU General Public License v2.0
709 stars 248 forks source link

Default script file size limit is too low #247

Closed meisterT closed 7 years ago

meisterT commented 7 years ago

Use c++11 and compile something like: int array[$large_num] = {-1};

This will create a large binary outside of DOMjudge and crash with an unintuitive gcc internal error in the sandbox.

a) report back, when the program is exactly on file size limit to give a better error message? b) increase defaults c) add config check

eldering commented 7 years ago

I already committed https://github.com/DOMjudge/domjudge/commit/223e31f43885a8ca5c14d7b5ab0d43b99fd532c1 Do you think that addresses the issue correctly? Feel free to make changes.

meisterT commented 7 years ago

I must have missed that, closing.