View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0026755 | Community | OCCT:Modeling Data | public | 2015-10-07 13:29 | 2016-04-20 15:49 |
Reporter | Assigned To | bugmaster | |||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Target Version | 7.0.0 | Fixed in Version | 7.0.0 | ||
Summary | 0026755: Use of reference to destroyed temporary object in Adaptor3d_SurfaceOfRevolution | ||||
Description | The problem has been reported and analyzed in OCE issue 559, see https://github.com/tpaviot/oce/issues/559#issuecomment-146139931 In Adaptor3d_SurfaceOfRevolution::GetType(), the following code uses local reference variable pointing to field of temporary object which gets destroyed immediately: const gp_Ax1& Axe = (myBasisCurve->Line()).Position(); | ||||
Steps To Reproduce | N/A | ||||
Tags | No tags attached. | ||||
Test case number | Not needed | ||||
|
Branch CR26755 has been created by abv. SHA-1: d550c521cdec99f252372f42c7f96a671e5c9121 Detailed log of new commits: Author: abv Date: Wed Oct 7 13:34:41 2015 +0300 0026755: Use of reference to destroyed temporary object in Adaptor3d_SurfaceOfRevolution Code corrected to make a copy of temporary object for its further use |
|
Fix pushed to CR26755, please review |
|
Reviewed. |
|
Dear BugMaster, Branch CR26755 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode. SHA-1: d550c521cdec99f252372f42c7f96a671e5c9121 Number of compiler warnings: occt component : Linux: 13 (13 on master) Windows: 0 (0 on master) products component : Linux: 39 (39 on master) Windows: 0 (0 on master) Regressions/Differences/Improvements: No regressions/differences Testing cases: Not needed Testing on Linux: occt component : Total MEMORY difference: 92382125 / 92366399 [+0.02%] Total CPU difference: 19692.759999999835 / 19714.639999999818 [-0.11%] products component : Total MEMORY difference: 26333558 / 26369733 [-0.14%] Total CPU difference: 7472.599999999996 / 7378.749999999995 [+1.27%] Testing on Windows: occt component : Total MEMORY difference: 58080389 / 58089039 [-0.01%] Total CPU difference: 18471.423205798783 / 18228.68564979895 [+1.33%] products component : Total MEMORY difference: 17178479 / 17183716 [-0.03%] Total CPU difference: 5938.380866299992 / 5689.09126829997 [+4.38%] There are no differences in images found by testdiff. |
|
Dear BugMaster, Branch CR26755 is TESTED. |
|
Branch CR26755 has been deleted by kgv. SHA-1: d550c521cdec99f252372f42c7f96a671e5c9121 |
occt: master 53dda4e9 2015-10-07 10:34:41
Committer: bugmaster Details Diff |
0026755: Use of reference to destroyed temporary object in Adaptor3d_SurfaceOfRevolution Code corrected to make a copy of temporary object for its further use |
Affected Issues 0026755 |
|
mod - src/Adaptor3d/Adaptor3d_SurfaceOfRevolution.cxx | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-10-07 13:29 |
|
New Issue | |
2015-10-07 13:29 |
|
Assigned To | => msv |
2015-10-07 13:35 | git | Note Added: 0046557 | |
2015-10-07 13:38 |
|
Note Added: 0046558 | |
2015-10-07 13:38 |
|
Status | new => resolved |
2015-10-07 13:38 |
|
Steps to Reproduce Updated | |
2015-10-07 15:08 |
|
Note Added: 0046563 | |
2015-10-07 15:08 |
|
Assigned To | msv => bugmaster |
2015-10-07 15:08 |
|
Status | resolved => reviewed |
2015-10-07 15:10 |
|
Assigned To | bugmaster => mkv |
2015-10-07 15:13 |
|
Test case number | => Not needed |
2015-10-08 12:56 |
|
Note Added: 0046591 | |
2015-10-08 12:56 |
|
Note Added: 0046592 | |
2015-10-08 12:56 |
|
Assigned To | mkv => bugmaster |
2015-10-08 12:56 |
|
Status | reviewed => tested |
2015-10-09 15:22 | bugmaster | Changeset attached | => occt master 53dda4e9 |
2015-10-09 15:22 | bugmaster | Status | tested => verified |
2015-10-09 15:22 | bugmaster | Resolution | open => fixed |
2015-10-16 16:58 | git | Note Added: 0047106 | |
2016-04-20 15:44 |
|
Fixed in Version | => 7.0.0 |
2016-04-20 15:49 |
|
Status | verified => closed |