ForkMaster/smesh
1
0
Fork 0
mirror of https://git.salome-platform.org/gitpub/modules/smesh.git synced 2025-02-15 10:03:08 +05:00
Code Issues Packages Projects Releases Wiki Activity

0021408: EDf 2006 SMESH: USe of DoubleNodesOnGroupBoundaries on 2D leads to crash

Browse Source 
In DoubleNodesOnGroupBoundaries(), throw in case of not volume group
This commit is contained in:
eap 2011-11-01 14:07:50 +00:00
parent 6e09f8b4ce
commit eea7a3b824
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/SMESH_I/SMESH_MeshEditor_i.cxx
View File

@ -5673,6 +5673,7 @@ CORBA::Boolean SMESH_MeshEditor_i::Make2DMeshFrom3D()
CORBA::Boolean SMESH_MeshEditor_i::DoubleNodesOnGroupBoundaries( const SMESH::ListOfGroups& theDomains, CORBA::Boolean SMESH_MeshEditor_i::DoubleNodesOnGroupBoundaries( const SMESH::ListOfGroups& theDomains,
CORBA::Boolean createJointElems ) CORBA::Boolean createJointElems )
throw (SALOME::SALOME_Exception)
{ {
initData(); initData();
@ -5686,8 +5687,10 @@ CORBA::Boolean SMESH_MeshEditor_i::DoubleNodesOnGroupBoundaries( const SMESH::Li
for ( int i = 0, n = theDomains.length(); i < n; i++ ) for ( int i = 0, n = theDomains.length(); i < n; i++ )
{ {
SMESH::SMESH_GroupBase_var aGrp = theDomains[ i ]; SMESH::SMESH_GroupBase_var aGrp = theDomains[ i ];
if ( !CORBA::is_nil( aGrp ) && ( aGrp->GetType() != SMESH::NODE ) ) if ( !CORBA::is_nil( aGrp ) /*&& ( aGrp->GetType() != SMESH::NODE )*/ )
{ {
if ( aGrp->GetType() != SMESH::VOLUME )
THROW_SALOME_CORBA_EXCEPTION("Not a volume group", SALOME::BAD_PARAM);
TIDSortedElemSet domain; TIDSortedElemSet domain;
domain.clear(); domain.clear();
domains.push_back(domain); domains.push_back(domain);

3
src/SMESH_I/SMESH_MeshEditor_i.hxx
View File

@ -713,7 +713,8 @@ public:
* @return TRUE if operation has been completed successfully, FALSE otherwise * @return TRUE if operation has been completed successfully, FALSE otherwise
*/ */
CORBA::Boolean DoubleNodesOnGroupBoundaries( const SMESH::ListOfGroups& theDomains, CORBA::Boolean DoubleNodesOnGroupBoundaries( const SMESH::ListOfGroups& theDomains,
CORBA::Boolean createJointElems ); CORBA::Boolean createJointElems )
throw (SALOME::SALOME_Exception);
/*! /*!
* \brief Double nodes on some external faces and create flat elements. * \brief Double nodes on some external faces and create flat elements.
* Flat elements are mainly used by some types of mechanic calculations. * Flat elements are mainly used by some types of mechanic calculations.