+ issue #1474: crash selecting constraint

FreeCAD · Mar 15, 2014 · b7cb49c · b7cb49c
1 parent 599331e
commit b7cb49c
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/Mod/Sketcher/App/Sketch.cpp b/src/Mod/Sketcher/App/Sketch.cpp
@@ -1868,6 +1868,9 @@ int Sketch::setDatum(int constrId, double value)
 
 int Sketch::getPointId(int geoId, PointPos pos) const
 {
+    // do a range check first
+    if (geoId < 0 || geoId >= Geoms.size())
+        return -1;
     switch (pos) {
     case start:
         return Geoms[geoId].startPointId;