MantisBT - Community
View Issue Details
0022899Community[OCCT] OCCT:Foundation Classespublic2012-01-11 15:342012-03-29 17:26
szy 
bugmaster 
normalminor 
closedfixed 
ALL
[OCCT] 6.5.2 
[OCCT] 6.5.3[OCCT] 6.5.3 
Test case is not required
0022899: Thread-safety in Standard_GUID
[OCC Forum] [Bug report (and fix)] Thread-safety in Standard_GUID (Roman Lygin) - http://www.opencascade.org/org/forum/thread_22538/. [^]

" Standard_GUID.cxx unnecessarily used static variables breaking thread-safety (even thread re-entrancy).
The fix with a regression test is enclosed. It can be applied to 6.5.0, 6.5.1, 6.5.2."
======================================
OCC: Patch makes replacing of corresponding static variables to local ones.
See C++ code for reproducing the problem in the attached archive, file Standard_GUIDTest.cxx
No tags attached.
zip fix515-standard_guid.zip (4,769) 2012-01-11 15:34
https://tracker.dev.opencascade.org/
Issue History
2012-01-11 15:34szyNew Issue
2012-01-11 15:34szyAssigned To => abv
2012-01-11 15:34szyFile Added: fix515-standard_guid.zip
2012-01-19 09:07abvStatusnew => assigned
2012-01-19 09:07abvDescription Updatedbug_revision_view_page.php?rev_id=3122#r3122
2012-01-19 09:44abvNote Added: 0019178
2012-01-19 09:44abvAssigned Toabv => kgv
2012-01-19 09:44abvStatusassigned => resolved
2012-01-19 09:45abvNote Edited: 0019178bug_revision_view_page.php?bugnote_id=19178#r3124
2012-01-19 09:50abvSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3126#r3126
2012-01-19 10:21kgvNote Added: 0019181
2012-01-19 10:21kgvStatusresolved => reviewed
2012-01-20 16:17mkvTest case number => Test case is not required
2012-01-20 17:08aanNote Added: 0019210
2012-01-20 17:15aanAssigned Tokgv =>
2012-01-20 17:15aanStatusreviewed => tested
2012-01-20 18:51abvNote Added: 0019215
2012-01-27 17:57bugmasterStatustested => verified
2012-01-27 17:57bugmasterResolutionopen => fixed
2012-01-27 17:57bugmasterAssigned To => Roman Lygin
2012-01-27 17:58bugmasterNote Added: 0019291
2012-03-29 17:26bugmasterChangeset attached => occt master 6b09c923

Notes
(0019178)
abv   
2012-01-19 09:44   
(edited on: 2012-01-19 09:45)
The code (original change + removal of some unnecessary variables around) integrated to branch OCC22899, please review and test.

(0019181)
kgv   
2012-01-19 10:21   
Bug branch passed review without remarks.
(0019210)
aan   
2012-01-20 17:08   
Dear BugMaster,
Workbench KAS:dev:mkv-22899-occt was created from SVN branch http://svn/svn/occt/branches/OCC22899 [^]
(and mkv-22899-products from trunk) and compiled on Linux platform.

There are not regressions in mkv-22899-products regarding to KAS:dev:products-20120117-opt

See results in /QADisk/occttests/results/KAS/dev/mkv-22899-products_19012012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120117-opt_17012012/lin
See test cases in /QADisk/occttests/tests/ED
N.B. In order to launch testing case you can make use the following instructions
http://doc/doku.php?id=occt:certification [^]
(0019215)
abv   
2012-01-20 18:51   
Dear Mikhail,

Test case IS required: please create a new test function for that case, see 'Steps to reproduce'
(0019291)
bugmaster   
2012-01-27 17:58   
Integration into trunk of occt repository

Date: 2012-01-27 17:43:57 +0400 (Fri, 27 Jan 2012)
New Revision: 10178

Modified:
   trunk/src/Standard/Standard_GUID.cxx