<html>
<head>
<base href="https://bugs.documentfoundation.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:s.mehrbrodt@gmail.com" title="Samuel Mehrbrodt (CIB) <s.mehrbrodt@gmail.com>"> <span class="fn">Samuel Mehrbrodt (CIB)</span></a>
</span> changed
<a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - Crash when pasting quotes with fixed width"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=131273">bug 131273</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>NEW
</td>
<td>ASSIGNED
</td>
</tr>
<tr>
<td style="text-align:right;">Assignee</td>
<td>libreoffice-bugs@lists.freedesktop.org
</td>
<td>serval2412@yahoo.fr
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - Crash when pasting quotes with fixed width"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=131273#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - Crash when pasting quotes with fixed width"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=131273">bug 131273</a>
from <span class="vcard"><a class="email" href="mailto:s.mehrbrodt@gmail.com" title="Samuel Mehrbrodt (CIB) <s.mehrbrodt@gmail.com>"> <span class="fn">Samuel Mehrbrodt (CIB)</span></a>
</span></b>
<pre>(In reply to Julien Nabet from <a href="show_bug.cgi?id=131273#c3">comment #3</a>)
<span class="quote">> Don't know if it's a naive patch but I don't reproduce the crash with it:
> diff --git a/sc/source/ui/docshell/impex.cxx
> b/sc/source/ui/docshell/impex.cxx
> index f99a8ce5fff5..56aa2e7f2506 100644
> --- a/sc/source/ui/docshell/impex.cxx
> +++ b/sc/source/ui/docshell/impex.cxx
> @@ -1278,7 +1278,7 @@ static OUString lcl_GetFixed( const OUString& rLine,
> sal_Int32 nStart, sal_Int32
> bool bFits = (nSpace - nStart - 3 <= SAL_MAX_UINT16);
> OSL_ENSURE( bFits, "lcl_GetFixed: line doesn't fit into data");
> if (bFits)
> - return rLine.copy(nStart+1, nSpace-nStart-2);
> + return rLine.copy(nStart+1, std::max(0, nSpace-nStart-2));
> else
> {
> rbOverflowCell = true;</span >
Just upload to gerrit. Thx!</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>