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

[Libreoffice Gerrit user]

 lotuswordpro/source/filter/tocread.cxx |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit efbd217afc52390138195e06e96ef9c8f10bf37b
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Tue Mar 5 13:25:16 2019 +0000
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Mar 5 22:09:26 2019 +0100

    Resolves: tdf#123815 null terminator included in string
    
    regression from...
    
    commit 18d636063fd7be165e7888af49372a6e2b851776
    Author: Caolán McNamara <caolanm at redhat.com>
    Date:   Mon Jun 19 09:46:31 2017 +0100
    
        ofz: use OString instead of bare char*
    
    this is no use to the reporter though, as the document format is
    10 and 11 is the oldest the filter will open
    
    Change-Id: I4cdfe92e37f047493a6eab95f0e8e9f5dced1a35
    Reviewed-on: https://gerrit.libreoffice.org/68745
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/lotuswordpro/source/filter/tocread.cxx b/lotuswordpro/source/filter/tocread.cxx
index e2421efb0931..203fe79a466f 100644
--- a/lotuswordpro/source/filter/tocread.cxx
+++ b/lotuswordpro/source/filter/tocread.cxx
@@ -290,7 +290,9 @@ CBenTOCReader::ReadTOC()
                         return Err;
                     }
 
-                    OString sName(sBuffer, Length);
+                    OString sName;
+                    if (Length)
+                        sName = OString(sBuffer, Length - 1);
 
                     CUtListElmt * pPrevNamedObjectListElmt;
                     if (FindNamedObject(&cpContainer->GetNamedObjects(),