[Tarantool-patches] [PATCH luajit 2/3] Fix potential file descriptor leak in luaL_loadfile*().

Sergey Kaplun skaplun at tarantool.org
Fri Jun 6 18:14:52 MSK 2025


Hi, Sergey!
Thanks for the review!

On 06.06.25, Sergey Bronnikov wrote:
> Hello, Sergey,
> 
> the test is passed when CMake option -DLUAJIT_USE_VALGRIND=ON is used and
> 
> patch with fix is reverted.

You should run it with the corresponding env variable (like it is done
in the CI), see the comment in the test header:

| VALGRIND_OPTS="--track-fds=yes" ctest -V -R lj-1249

> 
> Sergey

-- 
Best regards,
Sergey Kaplun


More information about the Tarantool-patches mailing list