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

[Caolán McNamara (via logerrit)]

 basctl/source/basicide/macrodlg.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 42a32f375a2641e6e82ee685efa3573c0dcaf001
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sun Aug 7 12:34:00 2022 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Sun Aug 7 15:37:14 2022 +0200

    tdf#150291 crash on macro organizer with no macro selected
    
    Change-Id: Ib7e05f4e3e1c63bc0bd497b590256c4ee77a0bf8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137922
    Tested-by: Caolán McNamara <caol...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/basctl/source/basicide/macrodlg.cxx 
b/basctl/source/basicide/macrodlg.cxx
index 03613c96c914..a9d3fd0f037c 100644
--- a/basctl/source/basicide/macrodlg.cxx
+++ b/basctl/source/basicide/macrodlg.cxx
@@ -122,7 +122,8 @@ MacroChooser::~MacroChooser()
 
 void MacroChooser::StoreMacroDescription()
 {
-    m_xBasicBox->get_selected(m_xBasicBoxIter.get());
+    if (!m_xBasicBox->get_selected(m_xBasicBoxIter.get()))
+        return;
     EntryDescriptor aDesc = 
m_xBasicBox->GetEntryDescriptor(m_xBasicBoxIter.get());
     OUString aMethodName;
     if (m_xMacroBox->get_selected(m_xMacroBoxIter.get()))