commit e6c07d2018d0b6be7492664fa22f70461b1422b8
author Caolán McNamara <caolanm@redhat.com> Mon Sep 23 15:38:12 2019 +0100
committer Xisco Faulí <xiscofauli@libreoffice.org> Tue Sep 24 09:29:59 2019 +0200
tree 119da8711d26024a96ce7458b6871f28ae81b6a9
parent d7bd0b2d010b4fa64dd6773ad63f864e13fffb29

Resolves: tdf#127695 IsGroupObject == true doesn't mean its a SdrObjGroup

but that it has a SubList

Change-Id: I473464de5c6bba5c36be1ad7cbd21a3158645d66
Reviewed-on: https://gerrit.libreoffice.org/79412
Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>
Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>

sw/source/core/doc/doclay.cxx

diff --git a/sw/source/core/doc/doclay.cxx b/sw/source/core/doc/doclay.cxx
index 4d8775f..851be694 100644
--- a/sw/source/core/doc/doclay.cxx
+++ b/sw/source/core/doc/doclay.cxx
@@ -1306,10 +1306,12 @@ static void lcl_SetNumUsedBit(std::vector<sal_uInt8>& rSetFlags, size_t nFormatS
    // contents for name collision check
    if (rObj.IsGroupObject())
    {
        const SdrObjGroup &rGroupObj = static_cast<const SdrObjGroup&>(rObj);
        for (size_t i = 0, nCount = rGroupObj.GetObjCount(); i < nCount; ++i)
        const SdrObjList* pSub(rObj.GetSubList());
        assert(pSub && "IsGroupObject is implented as GetSubList != nullptr");
        const size_t nCount = pSub->GetObjCount();
        for (size_t i = 0; i < nCount; ++i)
        {
            SdrObject* pObj = rGroupObj.GetObj(i);
            SdrObject* pObj = pSub->GetObj(i);
            if (!pObj)
                continue;
            lcl_SetNumUsedBit(rSetFlags, nFormatSize, nNmLen, *pObj, rCmpName);