<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p><br>
</p>
<div class="moz-cite-prefix">On 10.01.2025 14:10, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite" cite="mid:Z4EAGw2sAggMI2Gi@root">
<pre class="moz-quote-pre" wrap="">Hi, Sergey!
Thanks for the review.
On 10.01.25, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Hi, Sergey,
thanks for the patch!
test is passed with reverted patch.
With original reproducer luajit segfaults.
CMake options: cmake -S . -B build -DCMAKE_BUILD_TYPE=Debug
-DLUA_USE_APICHECK=ON -DLUA_USE_ASSERT=ON
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
You must run this test with Valgrind enabled to see the failure as it is
mentioned in the commit.</pre>
</blockquote>
<p><br>
</p>
<p>cmake arguments required for reproducing a bug by proposed test:</p>
<p> cmake -S . -B build -DCMAKE_BUILD_TYPE=Debug
-DLUA_USE_APICHECK=ON -DLUA_USE_ASSERT=ON -DLUAJIT_USE_VALGRIND=ON
-DLUAJIT_ENABLE_GC64=ON -DLUAJIT_USE_SYSMALLOC=ON</p>
<p><br>
</p>
<blockquote type="cite" cite="mid:Z4EAGw2sAggMI2Gi@root">
<pre class="moz-quote-pre" wrap="">The original reproducer is clumsy and unstable
-- any slight change of the parser may break it. This reproducer is much
simpler, robust, and more readable.
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">
Sergey
On 09.01.2025 18:01, Sergey Kaplun wrote:
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
<snipped>
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">+-- The parsing of the `for` iterator uses the incorrect check for
+-- `fs->bclim`, which allows the usage of an uninitialized value,
+-- so the test fails under Valgrind.
</pre>
</blockquote>
</blockquote>
<pre class="moz-quote-pre" wrap="">
<snipped>
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>