<div dir="ltr"><div>This one is mine, it was a crash apol got.</div><div>not bidi related.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 9, 2022 at 5:33 PM Kurt Hindenburg <<a href="mailto:kurt.hindenburg@gmail.com">kurt.hindenburg@gmail.com</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">FYI - master today - I thought a fix went in for the bidi issues?<br>
<br>
Analysis Summary:<br>
     New defects found: 1<br>
     Defects eliminated: 0<br>
<br>
Kurt<br>
<br>
> Begin forwarded message:<br>
> <br>
> 1 new defect(s) introduced to Konsole found with Coverity Scan.<br>
> <br>
> <br>
> New defect(s) Reported-by: Coverity Scan<br>
> Showing 1 of 1 defect(s)<br>
> <br>
> <br>
> ** CID 356762:  Error handling issues  (CHECKED_RETURN)<br>
> /home/kurthindenburg/Devel/KDE/src/konsole/src/Screen.cpp: 1670 in Konsole::Screen::setSelectionEnd(int, int, bool)()<br>
> <br>
> <br>
> ________________________________________________________________________________________________________<br>
> *** CID 356762:  Error handling issues  (CHECKED_RETURN)<br>
> /home/kurthindenburg/Devel/KDE/src/konsole/src/Screen.cpp: 1670 in Konsole::Screen::setSelectionEnd(int, int, bool)()<br>
> 1664     <br>
> 1665             // This should never happen, but it's happening. this is just to gather some information<br>
> 1666             // about the crash.<br>
> 1667             // Do not let this code go to a release.<br>
> 1668             if (_screenLines.size() < line) {<br>
> 1669                 QFile konsoleInfo(QStringLiteral("~/konsole_info_crash_array_out_of_bounds.txt"));<br>
>>>>   CID 356762:  Error handling issues  (CHECKED_RETURN)<br>
>>>>   Calling "open" without checking return value (as is done elsewhere 11 out of 12 times).<br>
> 1670                 konsoleInfo.open(QIODevice::WriteOnly);<br>
> 1671                 QTextStream messages(&konsoleInfo);<br>
> 1672     <br>
> 1673                 messages << "_selBegin" << _selBegin << "\n";<br>
> 1674                 messages << "endPos" << endPos << "\n";<br>
> 1675                 messages << "_selBottomRight" << _selBottomRight << "\n";<br>
> <br>
> <br>
<br>
</blockquote></div>