-
Daniel Vetter authored
Some tests assume that close on an unopened fd is fine, except it's not just unopened but actually stack garbage. There's a good chance we end up running close(0), which wreaks the testcase enumeration. Fix this. Reviewed-by: Jani Nikula <jani.nikula@intel.com> Not sure there's a better way to catch this than git grep plus lots of manual auditing ... The unit tests only catch it by accident (x86 works fine, I hit it in one testcase only cross-building to aarch64). v2: Note that gem_pread_after_blit uses heap memory, so is actually guaranteed to run close(0). Noticed by Jani. Reviewed-by: Jani Nikula <jani.nikula@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
de775f3a