Merge pull request #4121 from everlanes/TechDraw_Segfault

[TD] Check return value of getMDIViewPage() a second time
FreeCAD · Dec 14, 2020 · 0e556a0 · 0e556a0
2 parents fe721a1 + a6e29e3
commit 0e556a0
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/Mod/TechDraw/Gui/AppTechDrawGuiPy.cpp b/src/Mod/TechDraw/Gui/AppTechDrawGuiPy.cpp
@@ -193,7 +193,11 @@ class Module : public Py::ExtensionModule<Module>
                        } else {
                            vpp->showMDIViewPage();
                            mdi = vpp->getMDIViewPage();
-                           mdi->printPdf(filePath);
+                           if (mdi) {
+                               mdi->printPdf(filePath);
+                           } else {
+                               throw Py::TypeError("Page not available! Is it Hidden?");
+                           }
                        }
                    }
                }
@@ -234,7 +238,11 @@ class Module : public Py::ExtensionModule<Module>
                        } else {
                            vpp->showMDIViewPage();
                            mdi = vpp->getMDIViewPage();
-                           mdi->saveSVG(filePath);
+                           if (mdi) {
+                               mdi->saveSVG(filePath);
+                           } else {
+                               throw Py::TypeError("Page not available! Is it Hidden?");
+                           }
                        }
                    }
                }