From: cristy <urban-warrior@git.imagemagick.org>
Date: Fri, 7 May 2010 01:38:41 +0000 (+0000)
Subject: (no commit message)
X-Git-Tag: 7.0.1-0~9545
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=fe12d6c09cc8e153c2e23cf5199a5deb85cd20e5;p=imagemagick

---

diff --git a/coders/bmp.c b/coders/bmp.c
index 0da5ca997..b54f2d786 100644
--- a/coders/bmp.c
+++ b/coders/bmp.c
@@ -1178,6 +1178,7 @@ static Image *ReadBMPImage(const ImageInfo *image_info,ExceptionInfo *exception)
             q->blue=ScaleCharToQuantum(*p++);
             q->green=ScaleCharToQuantum(*p++);
             q->red=ScaleCharToQuantum(*p++);
+            SetOpacityPixelComponent(q,OpaqueOpacity);
             q++;
           }
           if (SyncAuthenticPixels(image,exception) == MagickFalse)
diff --git a/magick/module.c b/magick/module.c
index 46976cbc4..85be72b50 100644
--- a/magick/module.c
+++ b/magick/module.c
@@ -988,6 +988,7 @@ MagickExport MagickBooleanType InvokeDynamicImageFilter(const char *tag,
   /*
     Execute the module.
   */
+  ClearMagickException(exception);
   image_filter=(ImageFilterHandler *) lt_dlsym(handle,name);
   if (image_filter == (ImageFilterHandler *) NULL)
     (void) ThrowMagickException(exception,GetMagickModule(),ModuleError,
@@ -1005,23 +1006,17 @@ MagickExport MagickBooleanType InvokeDynamicImageFilter(const char *tag,
         (void) LogMagickEvent(ModuleEvent,GetMagickModule(),"\"%s\" completes",
           tag);
       if (signature != MagickImageFilterSignature)
-        {
-          (void) ThrowMagickException(exception,GetMagickModule(),ModuleError,
-            "ImageFilterSignatureMismatch","`%s': %8lx != %8lx",tag,signature,
-            MagickImageFilterSignature);
-          return(MagickFalse);
-        }
+        (void) ThrowMagickException(exception,GetMagickModule(),ModuleError,
+          "ImageFilterSignatureMismatch","`%s': %8lx != %8lx",tag,signature,
+          MagickImageFilterSignature);
     }
   /*
     Close the module.
   */
   if (lt_dlclose(handle) != 0)
-    {
-      (void) ThrowMagickException(exception,GetMagickModule(),ModuleWarning,
-        "UnableToCloseModule","`%s': %s",name,lt_dlerror());
-      return(MagickFalse);
-    }
-  return(MagickTrue);
+    (void) ThrowMagickException(exception,GetMagickModule(),ModuleWarning,
+      "UnableToCloseModule","`%s': %s",name,lt_dlerror());
+  return(exception->severity < ErrorException ? MagickTrue : MagickFalse);
 }
 
 /*