1) Crash on Linux study closing

2) Close .fcntl file before removing to delete study folder correctly

diff --git a/src/SUIT/SUIT_Session.cxx b/src/SUIT/SUIT_Session.cxx
index 9ab60b1ab43a5e4126512facad90f121c612b481..215416c870ecd554715db3d6681843b43542dd7c 100755 (executable)
--- a/src/SUIT/SUIT_Session.cxx
+++ b/src/SUIT/SUIT_Session.cxx
@@ -83,6 +83,12 @@ SUIT_Session::~SUIT_Session()
   // remove backup
   if ( myBFile )
     fclose( myBFile );
+
+#ifndef WIN32
+  if ( myBFileFcntl )
+    fclose( myBFileFcntl );
+#endif
+
   if ( !myBFolder.isEmpty() )
     Qtx::rmDir( myBFolder );
 
@@ -576,14 +582,20 @@ int SUIT_Session::lockFcntl( QString theLF )
   if ( theLF.isEmpty() )
     aFD = myBFileFcntl;
   else
+  {
     aFD = fopen( theLF.toLatin1().constData(), "w" );
+    if ( !aFD )
+      return -1;
+  }
   
   struct flock fLock;
   fLock.l_type = F_WRLCK;
   fLock.l_whence = SEEK_SET;
   fLock.l_len = 0;
   fLock.l_start = 0;
-  return fcntl( fileno( aFD ), F_SETLK, &fLock );
+
+  int ret = fcntl( fileno( aFD ), F_SETLK, &fLock );
+  return ret;
 }
 
 #endif
@@ -787,7 +799,13 @@ void SUIT_Session::removeTmpFiles( const bool withBackup )
       if ( !QFileInfo( blocName ).exists() || !locked  )
       {
         // unused non-removed folder
+#ifdef WIN32
         Qtx::rmDir( currF );
+#else 
+        // use the system call Qtx::rmDir() may not work on Linux
+        QString cmd = QString( "rm -rf " ) + currF.toLatin1().constData();
+        system( cmd.toLatin1().constData() );
+#endif
       }
     }
   }