From: abv <abv@opencascade.com>
Date: Wed, 19 Feb 2014 04:54:49 +0000 (+0400)
Subject: 0024640: Broken logic of check of variable for zero in IntCurvesFace_Intersector... 
X-Git-Tag: V6_8_0_beta~535
X-Git-Url: http://git.dev.opencascade.org/gitweb/?a=commitdiff_plain;h=7eb732b68317535c351110bae3e707bba820a1ab;p=occt-copy.git

0024640: Broken logic of check of variable for zero in IntCurvesFace_Intersector constructor

The check is fixed to avoid FPE division by zero
---

diff --git a/src/IntCurvesFace/IntCurvesFace_Intersector.cxx b/src/IntCurvesFace/IntCurvesFace_Intersector.cxx
index 4a1c6809de..0f93db935f 100644
--- a/src/IntCurvesFace/IntCurvesFace_Intersector.cxx
+++ b/src/IntCurvesFace/IntCurvesFace_Intersector.cxx
@@ -95,7 +95,7 @@ IntCurvesFace_Intersector::IntCurvesFace_Intersector(const TopoDS_Face& Face,
     Standard_Boolean bFlag;
     //
     {
-      Standard_Real dU, dV, dA, dB, aR, aTresh; 
+      Standard_Real dU, dV, dA, dB, aTresh; 
       bFlag=Standard_True;
       //
       aTresh=100.;
@@ -108,15 +108,9 @@ IntCurvesFace_Intersector::IntCurvesFace_Intersector(const TopoDS_Face& Face,
 	dB=dU;
       }
       //
-      aR=dA/dB;
-      if (dB<Precision::PConfusion()) {
+      if (dB < Precision::PConfusion() || dA > dB * aTresh) {
 	bFlag=!bFlag;
       }
-      else {
-	if (aR>aTresh) {
-	  bFlag=!bFlag;
-	}
-      }
     }
     //
     if (bFlag) {