* #6392: fix memory leak (opendir() without closedir())

Also remove 2 levels of indentation to make to code more readable

bug found by cppcheck

sflphone-common/src/plug-in/pluginmanager.cpp

@@ -87,14 +87,18 @@ PluginManager::loadPlugins (const std::string &path)
     dir = opendir (pluginDir.c_str());
     /* Test if the directory exists or is readable */
 
-    if (dir) {
+    if (!dir)
+        return 1;
+
     /* Read the directory */
     while ( (dirStruct=readdir (dir))) {
         /* Get the name of the current item in the directory */
         current = dirStruct->d_name;
+
         /* Test if the current item is not the parent or the current directory and that it ends with .so*/
+        if (current == pDir || current == cDir || !hasSharedExtension(current))
+            continue;
 
-            if (current != pDir && current != cDir and hasSharedExtension(current)) {
 
         /* Load the dynamic library */
         library = loadDynamicLibrary (pluginDir + current);
@@ -103,18 +107,17 @@ PluginManager::loadPlugins (const std::string &path)
 
         if (instanciatePlugin (library, &plugin) != 0) {
             _debug ("Error instanciating the plugin ...");
+            closedir(dir);
             return 1;
         }
 
         /* Regitering the current plugin */
         if (registerPlugin (plugin, library) != 0) {
             _debug ("Error registering the plugin ...");
+            closedir(dir);
             return 1;
         }
     }
-        }
-    } else
-        return 1;
 
     /* Close the directory */
     closedir (dir);