Archived issue #0022092
Crash of application on attempt to load a VRML file with all degenerated triangles
Description
In case of a VRML file contains all triangles degenerated, the application
crashes... Certainly, the VRML file is invalid. But at least a simple protection
should be added to Open CASCADE.
crashes... Certainly, the VRML file is invalid. But at least a simple protection
should be added to Open CASCADE.
Public activity
12 archived notes
Participants are labeled by their role within this record.
Dear Anton,
Please prepare branch from Attachment 3 (ZIP) archive and make revising
Please prepare branch from Attachment 3 (ZIP) archive and make revising
The SVN branch http://svn/svn/occt/branches/OCC22092 has been merged with fix "vro-OCC22092-v1"
Dear Commenter 1,
The branch has been reviewed and ready for tests.
The branch has been reviewed and ready for tests.
Dear Commenter 1,
Workbench KAS:dev:mkv-22092-occt was created from SVN branch http://svn/svn/occt/branches/OCC22092
(and mkv-22092-products from trunk) and compiled on Linux platform.
There are not regressions in mkv-22092-products regarding to KAS:dev:products-22122011-opt
See results in /QADisk/occttests/results/KAS/dev/mkv-22092-products_22122011/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20111216-opt_17122011/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
Workbench KAS:dev:mkv-22092-occt was created from SVN branch http://svn/svn/occt/branches/OCC22092
(and mkv-22092-products from trunk) and compiled on Linux platform.
There are not regressions in mkv-22092-products regarding to KAS:dev:products-22122011-opt
See results in /QADisk/occttests/results/KAS/dev/mkv-22092-products_22122011/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20111216-opt_17122011/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
Dear vro,
It necessary to create test case (new draw command) for testing this fix.
It necessary to create test case (new draw command) for testing this fix.
The draw-command exists: it is loadvrml.
Vlad,
It is necessary to create prototupe of testing case with usage of loadvrml command.
It is necessary to create prototupe of testing case with usage of loadvrml command.
loadvrml S sk97.wrl
The testing command is added.
Mikhail,
Please create testing case and change status to tested
Please create testing case and change status to tested
Dear Commenter 1,
Test case for this fix is chl/934/R1. It is OK.
Test case for this fix is chl/934/R1. It is OK.
Integrated into trunk of occt repository
Date: 2012-02-10 13:43:18 +0400 (Fri, 10 Feb 2012)
New Revision: 10405
Modified:
trunk/src/VrmlData/VrmlData_Group.cxx
trunk/src/VrmlData/VrmlData_IndexedFaceSet.cxx
trunk/src/VrmlData/VrmlData_Scene.cxx
Date: 2012-02-10 13:43:18 +0400 (Fri, 10 Feb 2012)
New Revision: 10405
Modified:
trunk/src/VrmlData/VrmlData_Group.cxx
trunk/src/VrmlData/VrmlData_IndexedFaceSet.cxx
trunk/src/VrmlData/VrmlData_Scene.cxx