old class Stack pop'ed 0 from empty stack, which std::stack doesn't

Some places in the code assumed that if the stack is empty a null pointer is
returned by top() (or old Pop()), this doesn't work anymore with ::std::stack
that instead has undefined behavior in that case, so check !stack.empty()
first before accessing top.
(cherry picked from commit ac40f7d6503533954127e818f2bf009200c1e3f2)

1 files changed, 8 insertions, 2 deletions

diff --git a/sc/source/filter/xml/xmlfilti.hxx b/sc/source/filter/xml/xmlfilti.hxx
index bb301013bdda..759f95468a00 100644
--- a/sc/source/filter/xml/xmlfilti.hxx
+++ b/sc/source/filter/xml/xmlfilti.hxx
@@ -262,8 +262,14 @@ public:
 
     void CloseConnection()
     {
-        bool bTemp = aConnectionOrStack.top();
-        aConnectionOrStack.pop();
+        bool bTemp;
+        if (aConnectionOrStack.empty())
+            bTemp = false;
+        else
+        {
+            bTemp = aConnectionOrStack.top();
+            aConnectionOrStack.pop();
+        }
         bConnectionOr = bTemp;
         bNextConnectionOr = bTemp;
     }