[Libreoffice-commits] core.git: sc/source

[Mike Kaganski (via logerrit)]

 sc/source/ui/docshell/impex.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 6155689bb6f1d72f29b43ac5ae94b32522ef9b42
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Fri May 21 00:03:43 2021 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Fri May 21 06:11:23 2021 +0200

    tdf#142395: properly handle "no string delimiter" case
    
    In this case, the delimiter character value would be '\0'.
    
    Change-Id: Idb0f01756991b3ea35a92f11b78fddd56c25265a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115839
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sc/source/ui/docshell/impex.cxx b/sc/source/ui/docshell/impex.cxx
index 3e58a9767cbe..b1a553a2ac10 100644
--- a/sc/source/ui/docshell/impex.cxx
+++ b/sc/source/ui/docshell/impex.cxx
@@ -1592,7 +1592,7 @@ const sal_Unicode* ScImportExport::ScanNextFieldFromString( const sal_Unicode* p
     rbIsQuoted = false;
     rField.clear();
     const sal_Unicode cBlank = ' ';
-    if (!ScGlobal::UnicodeStrChr( pSeps, cBlank))
+    if (cStr && !ScGlobal::UnicodeStrChr(pSeps, cBlank))
     {
         // Cope with broken generators that put leading blanks before a quoted
         // field, like "field1", "field2", "..."
@@ -1603,7 +1603,7 @@ const sal_Unicode* ScImportExport::ScanNextFieldFromString( const sal_Unicode* p
         if (*pb == cStr)
             p = pb;
     }
-    if ( *p == cStr )           // String in quotes
+    if (cStr && *p == cStr) // String in quotes
     {
         rbIsQuoted = true;
         const sal_Unicode* p1;