<div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div style="margin:0;">Hello,</div><div style="margin:0;"><br></div><div style="margin:0;">I found there some funny code in the function <b>x264_slicetype_analyse</b>.</div><div style="margin:0;">For example,</div><div style="margin:0;"><br></div><div style="margin:0;">...</div><div style="margin:0;"><div style="margin:0;">    else if( num_frames == 0 )</div><div style="margin:0;">    {</div><div style="margin:0;"><b>        frames[1]->i_type = X264_TYPE_I;</b></div><div style="margin:0;">        return;</div><div style="margin:0;">    }</div><div>...</div><div><div>    /* Restore frametypes for all frames that haven't actually been decided yet. */</div><div>    for( int j = reset_start; j <= num_frames; j++ )</div><div><b>        frames[j]->i_type = frames[j]->i_forced_type;</b></div></div><div>...</div><div><br></div><div><div>In here, the frames[] is temporary Stack allocated array variant, why we setting it's value before return?</div></div><div><br></div><div>Regards,</div><div>Min Chen</div><div><br></div></div></div>