

<HTML><BODY><div class="cl-fvacg5zsyy"><div>Hi, Sergey!</div><div> </div><div>Thanks for updating the test!<br><br>LGTM.</div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Evgeniy Temirgaleev</div></div></div><br><div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><span>From: Sergey Kaplun <<a href="mailto:skaplun@tarantool.org">skaplun@tarantool.org</a>><br>To: Evgeniy Temirgaleev <<a href="mailto:e.temirgaleev@tarantool.org">e.temirgaleev@tarantool.org</a>><br>Cc: tarantool-patches@dev.tarantool.org, Sergey Bronnikov <<a href="mailto:sergeyb@tarantool.org">sergeyb@tarantool.org</a>><br>Date: Wednesday, July 29, 2026 8:04 PM +03:00</span><br> <div><div id=""><div class="cl-e32954hmx9"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div id="style_17853446461606675592_mr_css_attr"><div id="style_17853446461606675592_BODY_mr_css_attr">Evgeniy,<br><br>On 29.07.26, Evgeniy Temirgaleev wrote:<br>> Hi, Sergey!<br>> Please, consider the idea about marking the tests with special build options.<br><br>Fixed your comment and force-pushed the branch.<br><br><snipped><br><br>><br>> I thinks it will be better to include such information to the test’s code in a some way.<br>> For example, we can add comments to the code, which we expect to fail with ASAN checks. Also, we can add mark to the corresponding test condition messages with ‘ASAN’ to reflect this in the test reports.<br><br>Good idea! I've added the ASAN tag in the test description as you<br>suggested and added the corresponding comment.<br><br>===================================================================<br>diff --git a/test/tarantool-tests/lj-1451-ffi-packed-bitfield.test.lua b/test/tarantool-tests/lj-1451-ffi-packed-bitfield.test.lua<br>index 76483b0d..8b4e5ce8 100644<br>--- a/test/tarantool-tests/lj-1451-ffi-packed-bitfield.test.lua<br>+++ b/test/tarantool-tests/lj-1451-ffi-packed-bitfield.test.lua<br>@@ -28,10 +28,13 @@ test:plan(5)<br><br>local packed = ffi.new('packed_struct')<br><br>-test:is(packed.bitfield, 0, 'correct 0-initialization')<br>+-- Check that there is no heap overflow for the packed FFI<br>+-- structure. That read/write access leads to the failure when<br>+-- LuaJIT is built with ASAN support.<br>+test:is(packed.bitfield, 0, 'ASAN: correct 0-initialization')<br><br>packed.bitfield = 1<br>-test:is(packed.bitfield, 1, 'bitfield set correctly')<br>+test:is(packed.bitfield, 1, 'ASAN: bitfield set correctly')<br><br>-- Check correct structure layout.<br>local byteoffset, bitpos, bitsize = ffi.offsetof('packed_struct2', 'bitfield16')<br>===================================================================<br><br>Branch is force-pushed.<br><br><snipped><br><br>><br>> --<br>> Best regards,<br>> Evgeniy Temirgaleev<br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></div></div></blockquote></div></div></BODY></HTML>