From b748cc688495500786f5f3a239bf0379916c915f Mon Sep 17 00:00:00 2001
From: Abdullah Tahiri <abdullah.tahiri.yo@gmail.com>
Date: Fri, 6 Mar 2020 16:47:42 +0100
Subject: [PATCH] Part: GeometryExtensions

========================

fixes #4087

Missing "break" in loop after finding the extension was causing segfault because of null pointer access, as the unique pointer had been std:move-d when found and was thus no longer valid.
---
 src/Mod/Part/App/Geometry.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Mod/Part/App/Geometry.cpp b/src/Mod/Part/App/Geometry.cpp
index adaa0ab3cf..5b0dff299d 100644
--- a/src/Mod/Part/App/Geometry.cpp
+++ b/src/Mod/Part/App/Geometry.cpp
@@ -316,6 +316,7 @@ void Geometry::setExtension(std::unique_ptr<GeometryExtension> && geo)
             ext->getName() == geo->getName()){
             ext = std::move(geo);
             hasext = true;
+            break;
         }
     }