[Libreoffice-commits] core.git: Branch 'libreoffice-7-2' - sc/source

[Eike Rathke (via logerrit)]

 sc/source/ui/docshell/impex.cxx |   11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

New commits:
commit d6808f6ba03f74f99759e8d59ac0a04fcc9c6dd8
Author:     Eike Rathke <erack at redhat.com>
AuthorDate: Mon Aug 16 21:46:44 2021 +0200
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Aug 17 10:51:59 2021 +0200

    Resolves: tdf#96561 Include embedded null-characters while assembling CSV line
    
    ... instead of prematurely ending analysis and skipping the
    remainder that is still part of data read.
    
    Change-Id: I8f34bc9672e37f9c1c1ae81ddeba464360add7c2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120555
    Reviewed-by: Eike Rathke <erack at redhat.com>
    Tested-by: Jenkins
    (cherry picked from commit 7bd03074ae7362d8fffe5182529626a0b43a0ed1)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120451
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sc/source/ui/docshell/impex.cxx b/sc/source/ui/docshell/impex.cxx
index 380bd875fadc..6297fba3a1a7 100644
--- a/sc/source/ui/docshell/impex.cxx
+++ b/sc/source/ui/docshell/impex.cxx
@@ -2515,8 +2515,17 @@ Label_RetryWithNewSep:
         while (!rStream.eof() && aStr.getLength() < nArbitraryLineLengthLimit)
         {
             const sal_Unicode * p = aStr.getStr() + nLastOffset;
-            while (*p)
+            const sal_Unicode * const pStop = aStr.getStr() + aStr.getLength();
+            while (p < pStop)
             {
+                if (!*p)
+                {
+                    // Skip embedded null-characters. They don't change
+                    // anything and are handled at a higher level.
+                    ++p;
+                    continue;
+                }
+
                 if (nQuotes)
                 {
                     if (*p == cFieldQuote)