MantisBT - Community
View Issue Details
0028888Community[OCCT] OCCT:Visualizationpublic2017-07-04 12:202017-09-29 16:28
Vico Liang 
bugmaster 
normaltrivial 
closedfixed 
WindowsVC++ 201364 bit
[OCCT] 5.2.2 
[OCCT] 7.2.0[OCCT] 7.2.0 
Not required
0028888: Visualization - AIS_InteractiveContext should not hold V3d_View handle which will cause accessing invalid removed V3d_View
AIS_InteractiveContext has a field: Handle(V3d_View) mylastmoveview;
This field is assigned in method MoveTo()

After MoveTo function is called, the view passed in as argument will be hold in field mylastmoveview. The problem will occur if we try to remove the view by calling view->Remove() operation. Normally, after view->Remove is called, the view's life should be end and it should be deleted from memory. But it won't due to that it's still referenced by AIS_InteractiveContext and then updating the viewer's structure, it will still reference to the invalid removed view.
No tags attached.
Issue History
2017-07-04 12:20Vico LiangNew Issue
2017-07-04 12:20Vico LiangAssigned To => kgv
2017-07-04 14:04kgvProduct Version7.1.0 => 5.2.2
2017-07-04 14:05kgvSummaryAIS_InteractiveContext should not hold V3d_View handle which will cause accessing invalid removed V3d_View => Visualization - AIS_InteractiveContext should not hold V3d_View handle which will cause accessing invalid removed V3d_View
2017-07-04 14:16gitNote Added: 0067972
2017-07-05 07:34kgvNote Added: 0067983
2017-07-05 07:34kgvAssigned Tokgv => bugmaster
2017-07-05 07:34kgvSeveritycrash => trivial
2017-07-05 07:34kgvStatusnew => resolved
2017-07-05 07:35kgvStatusresolved => reviewed
2017-07-05 11:17apvAssigned Tobugmaster => apv
2017-07-05 11:37apvTest case number => Not required
2017-07-05 13:39apvNote Added: 0068006
2017-07-05 13:39apvAssigned Toapv => bugmaster
2017-07-05 13:39apvStatusreviewed => tested
2017-07-05 13:40apvNote Added: 0068007
2017-07-07 17:25bugmasterChangeset attached => occt master 4679d975
2017-07-07 17:25bugmasterStatustested => verified
2017-07-07 17:25bugmasterResolutionopen => fixed
2017-07-10 08:58gitNote Added: 0068131
2017-09-29 16:18aivFixed in Version => 7.2.0
2017-09-29 16:28aivStatusverified => closed

Notes
(0067972)
git   
2017-07-04 14:16   
Branch CR28888 has been created by kgv.

SHA-1: 82bae867ee54b97350faef6256d9ba7191268679


Detailed log of new commits:

Author: kgv
Date: Tue Jul 4 14:15:34 2017 +0300

    0028888: Visualization - AIS_InteractiveContext should not hold V3d_View handle which will cause accessing invalid removed V3d_View
(0067983)
kgv   
2017-07-05 07:34   
Please take the patch.

CR28888-master-KGV-Products-Debian70-64-opt-tests-compare-linux - Build # 1 - Successful:
Check console output at http://jenkins-test-10.nnov.opencascade.com:8080/job/CR28888-master-KGV-Products-Debian70-64-opt-tests-compare-linux/1/ [^] to view the results.

CR28888-master-KGV-Products-Windows-64-VC10-opt-tests-compare-windows - Build # 1 - Successful:
Check console output at http://jenkins-test-10.nnov.opencascade.com:8080/job/CR28888-master-KGV-Products-Windows-64-VC10-opt-tests-compare-windows/1/ [^] to view the results.
(0068006)
apv   
2017-07-05 13:39   
Dear BugMaster,

Branch CR28888 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
SHA-1: 82bae867ee54b97350faef6256d9ba7191268679

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

Regressions/Differences:
Not detected

Testing cases:
Not required

Testing on Linux:
Total MEMORY difference: 92225898 / 92167247 [+0.06%]
Total CPU difference: 19126.850000000268 / 18926.030000000348 [+1.06%]

Testing on Windows:
Total MEMORY difference: 58847569 / 58850236 [-0.00%]
Total CPU difference: 17910.506010198515 / 17597.770805498705 [+1.78%]
(0068007)
apv   
2017-07-05 13:40   
Dear BugMaster,

Please integrate branch CR28888 into the occt git-repository master.
(0068131)
git   
2017-07-10 08:58   
Branch CR28888 has been deleted by kgv.

SHA-1: 82bae867ee54b97350faef6256d9ba7191268679