Fix a crash if SALOME module sources item does not have a pair of corresponding SALOM...

diff --git a/bin/SALOME_InstallWizard b/bin/SALOME_InstallWizard
index b6accf9aa04cf4c0ca397924399e1f2496497feb..c605690532370d4c1c489f4e86293711827905a1 100755 (executable)
Binary files a/bin/SALOME_InstallWizard and b/bin/SALOME_InstallWizard differ

diff --git a/src/SALOME_InstallWizard.cxx b/src/SALOME_InstallWizard.cxx
index bca0109e3c6313934dbe9e5e166f2c27a41415fd..583380cea668ad0d40372464bea627e9f20d1920 100644 (file)
--- a/src/SALOME_InstallWizard.cxx
+++ b/src/SALOME_InstallWizard.cxx
@@ -1835,7 +1835,7 @@ void SALOME_InstallWizard::reject()
                                   tr( "&Yes" ),
                                   tr( "&No" ),
                                   QString::null,
-                                  0,
+                                  1,
                                   1 ) == 1 ) {
       return;
     }

diff --git a/src/SALOME_ProductsView.cxx b/src/SALOME_ProductsView.cxx
index 5268830d2d86ddcebfbec7de780f90e708db73a2..c68ae029922c408f92fa9bbbea6dd43284858247 100644 (file)
--- a/src/SALOME_ProductsView.cxx
+++ b/src/SALOME_ProductsView.cxx
@@ -129,6 +129,8 @@ void ProductsView::itemActivated( QCheckListItem* item ) {
 // ================================================================
 bool ProductsView::isBinaries( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -147,6 +149,8 @@ bool ProductsView::isBinaries( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::isSources( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -168,6 +172,8 @@ bool ProductsView::isSources( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::isNative( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 && item->isEnabled() ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -196,6 +202,8 @@ bool ProductsView::isNone(  QCheckListItem* item ) const
  */
 // ================================================================
 void ProductsView::setBinaries( QCheckListItem* item ) {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* srcItem = 0;
@@ -228,6 +236,8 @@ void ProductsView::setBinaries( QCheckListItem* item ) {
  */
 // ================================================================
 void ProductsView::setSources( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* binItem = 0;
@@ -260,6 +270,8 @@ void ProductsView::setSources( QCheckListItem* item )  {
  */
 // ================================================================
 void ProductsView::setNative( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     QCheckListItem* binItem = 0;
@@ -291,6 +303,8 @@ void ProductsView::setNative( QCheckListItem* item )  {
  */
 // ================================================================
 void ProductsView::setNone( QCheckListItem* item )  {
+  if ( !item ) 
+    return;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -314,6 +328,8 @@ void ProductsView::setNone( QCheckListItem* item )  {
 // ================================================================
 bool ProductsView::hasBinaries( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -333,6 +349,8 @@ bool ProductsView::hasBinaries( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::hasSources( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -352,6 +370,8 @@ bool ProductsView::hasSources( QCheckListItem* item ) const
 // ================================================================
 bool ProductsView::hasNative( QCheckListItem* item ) const
 {
+  if ( !item ) 
+    return false;
   if ( item->childCount() > 0 ) {
     QCheckListItem* subItem = (QCheckListItem*)( item->firstChild() );
     while( subItem ) {
@@ -371,6 +391,8 @@ bool ProductsView::hasNative( QCheckListItem* item ) const
 // ================================================================
 void ProductsView::setItemEnabled( QCheckListItem* item, const bool enable )
 {
+  if ( !item ) 
+    return;
   item->setEnabled( enable );
   QListViewItem* subItem = item->firstChild();
   while( subItem ) {