0024517: Crash at StepToTopoDS_TranslateEdge::Init()

ID	Project	Category	View Status	Date Submitted	Last Update
0024517	Community	OCCT:Data Exchange	public	2014-01-08 12:38	2014-05-05 13:37

Reporter	gdumonteil	Assigned To	bugmaster
Priority	urgent	Severity	crash
Status	closed	Resolution	fixed
Platform	x64	OS	Windows
Product Version	6.7.0
Target Version	6.7.1	Fixed in Version	6.7.1

Summary	0024517: Crash at StepToTopoDS_TranslateEdge::Init()
Description	Issue 0023938 is parent. The first crash of issue 0023938 is corrected but the second is still crashing our application. The crash is at "StepToTopoDS_TranslateEdge::Init()", at line 308, when testing the kind of C: "if ( C->IsKind(STANDARD_TYPE(StepGeom_Pcurve)))" We correct here also by cheking the nullity of C before calling the IsKind function: if (!C.IsNull()) { if ( C->IsKind(STANDARD_TYPE(StepGeom_Pcurve))) { With this, we're finally able to fully transfer the STEP file.
Steps To Reproduce	We can't share the STEP file from our client. The STEP file was exported from SolidWorks 2011.
Tags	No tags attached.

Test case number	Not needed

~~gka~~ 2014-01-17 13:08 developer ~0027535	Branch CR24517 is ready to be reviewed.

~~abv~~ 2014-01-17 16:04 manager ~0027539	No remarks, please test

~~mkv~~ 2014-01-22 11:06 tester ~0027588	Dear BugMaster, Branch CR24517 (and products from GIT master) was compiled on Linux and Windows platforms and tested. SHA-1: 147098f646f964d17361aed67f274718187ca385 Number of compiler warnings: occt component : Linux: 48 (48 on master) Windows: 1 (1 on master) products component : Linux: 12 (12 on master) Windows: 2 (2 on master) Regressions/Differences: No regressions/differences Testing cases: Not needed Testing on Linux: Total MEMORY difference: 365409044 / 365937044 Total CPU difference: 43525.38000000006 / 44020.780000000144 Testing on Windows: Total MEMORY difference: 417225024 / 417459436 Total CPU difference: 32393.828125 / 32496.375 There are not differences in images found by testdiff.

Date Modified	Username	Field	Change
2014-01-08 12:38	gdumonteil	New Issue
2014-01-08 12:38	gdumonteil	Assigned To	=> gka
2014-01-17 11:40	~~gka~~	Status	new => assigned
2014-01-17 13:08	~~gka~~	Note Added: 0027535
2014-01-17 13:08	~~gka~~	Assigned To	gka => abv
2014-01-17 13:08	~~gka~~	Status	assigned => resolved
2014-01-17 16:04	~~abv~~	Note Added: 0027539
2014-01-17 16:04	~~abv~~	Assigned To	abv => bugmaster
2014-01-17 16:04	~~abv~~	Status	resolved => reviewed
2014-01-20 17:48	~~mkv~~	Assigned To	bugmaster => mkv
2014-01-22 11:05	~~mkv~~	Relationship added	child of 0023938
2014-01-22 11:06	~~mkv~~	Note Added: 0027588
2014-01-22 11:07	~~mkv~~	Test case number	=> Not needed
2014-01-22 11:07	~~mkv~~	Assigned To	mkv => bugmaster
2014-01-22 11:07	~~mkv~~	Status	reviewed => tested
2014-01-23 10:14	bugmaster	Target Version	=> 6.7.1
2014-01-24 15:35	bugmaster	Changeset attached	=> occt master 545ef510
2014-01-24 15:35	bugmaster	Status	tested => verified
2014-01-24 15:35	bugmaster	Resolution	open => fixed
2014-05-05 13:34	~~aiv~~	Status	verified => closed
2014-05-05 13:37	~~aiv~~	Fixed in Version	=> 6.7.1

occt: master 545ef510 2014-01-23 07:44:07 ~~gka~~ Committer: bugmaster Details Diff	0024517: Crash at StepToTopoDS_TranslateEdge::Init() Fixed crash at StepToTopoDS_TranslateEdge::Init() for case when curve is absent) Corrected error message for case of absence geom curve in edge curve		Affected Issues 0024517
	mod - src/StepToTopoDS/StepToTopoDS_TranslateEdge.cxx		Diff File