From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 36D1913CF8CB; Thu, 5 Jun 2025 08:44:18 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 36D1913CF8CB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1749102258; bh=46g9FycHn0IQy8rblbPy+11ydNakrwGb7KOsaZHYx6o=; h=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:Cc:From; b=bS8rxf9Hr2oioTSvA5pzH7xOErPb4UOB8objrTtUZWfzpvLUyi7gbPNRbNhxxpjm2 ldWMlXcHKf9l8+JFavRtmGkiOUefIphu+/Fey8YbGDjjEJhIDec5udOjIntR65iiU+ dZoFRa2nM8YHA6O/9H0YSLLaiQAYsT4GkCgAMndo= Received: from send279.i.mail.ru (send279.i.mail.ru [95.163.59.118]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 22454461F00 for ; Thu, 5 Jun 2025 08:44:16 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 22454461F00 Received: by exim-smtp-75656d46d5-jh4vp with esmtpa (envelope-from ) id 1uN3Oh-00000000FoO-0tve; Thu, 05 Jun 2025 08:44:15 +0300 To: Sergey Bronnikov Date: Thu, 5 Jun 2025 08:44:17 +0300 Message-ID: X-Mailer: git-send-email 2.49.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Mailru-Src: smtp X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD9851146C857904EAB2F2B7E44D6C9CDC5764826102E32DC0A182A05F53808504055E518F6C17BB51B3DE06ABAFEAF67058782C14A38F22AD995569FA2C0CFBECA15B839EAEBA98067 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7BC08626EA5717D14EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637AC83A81C8FD4AD23D82A6BABE6F325AC2E85FA5F3EDFCBAA7353EFBB5533756680FD86C7FE90CFE5D6ACFAEC698F39A4975AAC8D183FCD646A19BDF68C6EF7FD389733CBF5DBD5E913377AFFFEAFD269176DF2183F8FC7C06030C3405640F6718941B15DA834481FCF19DD082D7633A0EF3E4896CB9E6436389733CBF5DBD5E9D5E8D9A59859A8B601F8F2FECC0250C8CC7F00164DA146DA6F5DAA56C3B73B237318B6A418E8EAB86D1867E19FE14079C09775C1D3CA48CF3D321E7403792E342EB15956EA79C166A417C69337E82CC275ECD9A6C639B01B78DA827A17800CE732FCE54C4D9A645443847C11F186F3C59DAA53EE0834AAEE X-C1DE0DAB: 0D63561A33F958A50BB851CB4FEC1BF45002B1117B3ED696F8723EC76DD89FC114DB8790748E3E77823CB91A9FED034534781492E4B8EEAD21D4E6D365FE45D1C79554A2A72441328621D336A7BC284946AD531847A6065A17B107DEF921CE79BDAD6C7F3747799A X-C8649E89: 1C3962B70DF3F0ADE00A9FD3E00BEEDF77DD89D51EBB7742D3581295AF09D3DF87807E0823442EA2ED31085941D9CD0AF7F820E7B07EA4CFFACDC66D2A75CE05D9EE458F77A180615EAD47427E95581331A6719D0D250B4D284AB208DE0D1BB14322DC41441C31F07B5E62DA5260F91F41DF049AB57831296F4DFD88F2F229975F4332CA8FE04980913E6812662D5F2A5EAB5682573093F7837F15F2B5E4A70B33F2C28C22F508233FCF178C6DD14203 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu53w8ahmwBjZKM/YPHZyZHvz5uv+WouB9+ObcCpyrx6l7KImUglyhkEat/+ysWwi0gdhEs0JGjl6ggRWTy1haxBpVdbIX1nthFXMZebaIdHP2ghjoIc/363UZI6Kf1ptIMVSykAyseJQ6/p77At0mPlz8= X-DA7885C5: 5FD2905955AAE2E3F255D290C0D534F9E396104AB93ED55BF4791D4989F1432B517184EBFD078A5D5B1A4C17EAA7BC4BEF2421ABFA55128DAF83EF9164C44C7E X-Mailru-Sender: 689FA8AB762F7393FE9E42A757851DB6482D958EC45BDEF387F66E1481AC81799BAE4412D9A6ED91E49D44BB4BD9522A059A1ED8796F048DB274557F927329BE89D5A3BC2B10C37545BD1C3CC395C826B4A721A3011E896F X-Mras: Ok Subject: [Tarantool-patches] [PATCH luajit 0/3] Fix descriptor leak in loadfile X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Sergey Kaplun via Tarantool-patches Reply-To: Sergey Kaplun Cc: tarantool-patches@dev.tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" This patch set fixes the possible descriptor leak in the `loadfile()` in case of the error. The first patch adds the Valgrind configuration to the corresponding CI workflow to test the descriptor leakage. The last two fix two similar scenarios of the leak. Branch: https://github.com/tarantool/luajit/tree/skaplun/lj-1249-loadfile-fd-leak Related issues: * https://github.com/LuaJIT/LuaJIT/issues/1249 * https://github.com/tarantool/tarantool/issues/11278 Mike Pall (2): Fix potential file descriptor leak in luaL_loadfile*(). Fix another potential file descriptor leak in luaL_loadfile*(). Sergey Kaplun (1): ci: add track-fds Valgrind scenario .github/workflows/valgrind-testing.yaml | 9 ++- src/lj_load.c | 19 +++--- .../lj-1249-loadfile-fd-leak.test.lua | 59 +++++++++++++++++++ test/tarantool-tests/utils/CMakeLists.txt | 1 + 4 files changed, 78 insertions(+), 10 deletions(-) create mode 100644 test/tarantool-tests/lj-1249-loadfile-fd-leak.test.lua -- 2.49.0