DiscussionsIssue archiveOCCT:Shape Healing

Archived issue #0026524

ShapeFix_Face crash when performing fix on an invalid face.

CommunityOCCT:Shape Healingclosed9 public notes

Search issues

Description

TopoDS_Face = // load from Attachment 1 (BREP)
ShapeFix_Face aFixFace(aFace);
aFixFace.Perform(); // Crash

Steps to reproduce

Branch CR26524 was ready to be reviewed

Public activity

9 archived notes

Participants are labeled by their role within this record.

01Author
The bad face was created using BRepBuilderAPI_MakeFace and then add a hole wire to the face. The problem is that the hole wire intersects with the boundary outer wire.
02Commenter 2
I cannot reproduce this issue in DRAW (tried OCCT 6.9.0, 6.9.1, and current master, vc10 64-bit Release)
03Commenter 2
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: gka
Date: Wed Oct 26 17:32:10 2016 +0300

    0026524: ShapeFix_Face crash when performing fix on an invalid face.
    
    Check that method Context() returns not null handle was added.
04Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
05Commenter 5
No remarks, please test
06Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
07Commenter 7
Branch [archived branch] has been rebased on the current master
08Commenter 8
Dear BugMaster,

Branch CR26524 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MasOS: 0 (0 on master)
products component:
   Linux: 63
   Windows: 0
   MacOS: 1125

Regressions/Differences:
Not detected

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 91277608 / 90743305 [+0.59%]
Total CPU difference: 19463.569999999916 / 19313.389999999898 [+0.78%]

Testing on Windows:
Total MEMORY difference: 57300773 / 57321116 [-0.04%]
Total CPU difference: 18001.173791398684 / 18220.18359529866 [-1.20%]
09Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]