https://bugs.documentfoundation.org/show_bug.cgi?id=90653

Julien Nabet <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #18 from Julien Nabet <[email protected]> ---
Tristan:the simplest thing would be to stop when there's a pb

Eike: do you think we may add some throw exception or at least an assert in
https://opengrok.libreoffice.org/xref/core/sc/source/ui/docshell/impex.cxx?r=f805e0b9#636
in else part?

626  static bool lcl_appendLineData( OUString& rField, const sal_Unicode* p1,
const sal_Unicode* p2 )
627  {
628      OSL_ENSURE( rField.getLength() + (p2 - p1) <= SAL_MAX_UINT16,
"lcl_appendLineData: data overflow");
629      if (rField.getLength() + (p2 - p1) <= SAL_MAX_UINT16)
630      {
631          rField += OUString( p1, sal::static_int_cast<sal_Int32>( p2 - p1 )
);
632          return true;
633      }
634      else
635      {
636          rField += OUString( p1, SAL_MAX_UINT16 - rField.getLength() );
637          return false;
638      }
639  }

Indeed, putting more than SAL_MAX_UINT16 characters in a cell is more than
suspect no ?

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to