<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Nov 15, 2019 at 12:35 PM Sigitas Tolusis <<a href="mailto:sigitas@vtex.lt">sigitas@vtex.lt</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Noticed two bugs.<br>
<br>
1. Mised case for user defined whatsit node type 108 (‘l’ Lua value)<br>
    in function check_node_wrapup_core (texnodes.c:2482).<br>
    Possible fix: just add the same as cases for types ’s’ and ‘d’.<br>
    Insert line at texnodes.c:2500<br>
                case ‘l':<br>
<br>
2. In function flush_node_wrapup_core (texnodes.c:2201) for user defined<br>
    whatsit node type ‘l’ we have: free_user_lua(user_node_value(p)).<br>
    But in luanode.c:409 free_user_lua is defined as:<br>
<br>
void free_user_lua(pointer p)<br>
{<br>
    if (user_node_value(p) != 0) {<br>
        luaL_unref(Luas, LUA_REGISTRYINDEX, user_node_value(p));<br>
    }<br>
}<br>
<br>
So we have two times user_node_value and it generates segmentation fault.<br>
Possible fix: free_user_lua(p); // (texnodes.c:2223)<br>
<br>
Regards,<br>
Sigitas<br></blockquote><div><br></div><div>Thank you for the report, we will see it asap.</div><div> </div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature">luigi<br></div></div>