[asterisk-commits] mnicholson: trunk r275144 - in /trunk: ./ main/loader.c

[SVN commits to the Asterisk project]

Author: mnicholson
Date: Fri Jul  9 12:50:45 2010
New Revision: 275144

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=275144
Log:
Merged revisions 275143 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r275143 | mnicholson | 2010-07-09 12:50:05 -0500 (Fri, 09 Jul 2010) | 2 lines
  
  don't unload modules that returned AST_MODULE_LOAD_DECLINE when they were loaded
........

Modified:
    trunk/   (props changed)
    trunk/main/loader.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/main/loader.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/loader.c?view=diff&rev=275144&r1=275143&r2=275144
==============================================================================
--- trunk/main/loader.c (original)
+++ trunk/main/loader.c Fri Jul  9 12:50:45 2010
@@ -467,7 +467,7 @@
 				continue;
 			}
 			AST_LIST_REMOVE_CURRENT(entry);
-			if (mod->info->unload) {
+			if (mod->flags.running && !mod->flags.declined && mod->info->unload) {
 				mod->info->unload();
 			}
 			AST_LIST_HEAD_DESTROY(&mod->users);
@@ -494,7 +494,7 @@
 		return -1;
 	}
 
-	if (!(mod->flags.running || mod->flags.declined))
+	if (!mod->flags.running || mod->flags.declined)
 		error = 1;
 
 	if (!error && (mod->usecount > 0)) {