]> granicus.if.org Git - imagemagick/commitdiff
...
authorCristy <urban-warrior@imagemagick.org>
Sat, 25 Aug 2018 18:17:07 +0000 (14:17 -0400)
committerCristy <urban-warrior@imagemagick.org>
Sat, 25 Aug 2018 18:17:07 +0000 (14:17 -0400)
MagickCore/constitute.c
MagickCore/token.c

index 620b33bcaefe682b5265d2f5a8d637f4c38bccb7..637cb1afb837b2b319ad9ed4cd373bbe52eaa2ef 100644 (file)
@@ -399,15 +399,14 @@ MagickExport Image *PingImages(ImageInfo *image_info,const char *filename,
 %
 */
 
-static MagickBooleanType IsCoderAuthorized(const char *module,const char *coder,
+static MagickBooleanType IsCoderAuthorized(const char *coder,
   const PolicyRights rights,ExceptionInfo *exception)
 {
-  if ((IsRightsAuthorized(ModulePolicyDomain,rights,module) == MagickFalse) ||
-      (IsRightsAuthorized(CoderPolicyDomain,rights,coder) == MagickFalse))
+  if (IsRightsAuthorized(CoderPolicyDomain,rights,coder) == MagickFalse)
     {
       errno=EPERM;
       (void) ThrowMagickException(exception,GetMagickModule(),PolicyError,
-        "NotAuthorized","`%s:%s'",module,coder);
+        "NotAuthorized","`%s'",coder);
       return(MagickFalse);
     }
   return(MagickTrue);
@@ -545,8 +544,7 @@ MagickExport Image *ReadImage(const ImageInfo *image_info,
       */
       if (GetMagickDecoderThreadSupport(magick_info) == MagickFalse)
         LockSemaphoreInfo(magick_info->semaphore);
-      status=IsCoderAuthorized(magick_info->module,read_info->magick,
-        ReadPolicyRights,exception);
+      status=IsCoderAuthorized(read_info->magick,ReadPolicyRights,exception);
       image=(Image *) NULL;
       if (status != MagickFalse)
         image=decoder(read_info,exception);
@@ -610,8 +608,7 @@ MagickExport Image *ReadImage(const ImageInfo *image_info,
       */
       if (GetMagickDecoderThreadSupport(magick_info) == MagickFalse)
         LockSemaphoreInfo(magick_info->semaphore);
-      status=IsCoderAuthorized(magick_info->module,read_info->magick,
-        ReadPolicyRights,exception);
+      status=IsCoderAuthorized(read_info->magick,ReadPolicyRights,exception);
       image=(Image *) NULL;
       if (status != MagickFalse)
         image=(decoder)(read_info,exception);
@@ -1168,8 +1165,7 @@ MagickExport MagickBooleanType WriteImage(const ImageInfo *image_info,
       */
       if (GetMagickEncoderThreadSupport(magick_info) == MagickFalse)
         LockSemaphoreInfo(magick_info->semaphore);
-      status=IsCoderAuthorized(magick_info->module,write_info->magick,
-        WritePolicyRights,exception);
+      status=IsCoderAuthorized(write_info->magick,WritePolicyRights,exception);
       if (status != MagickFalse)
         status=encoder(write_info,image,exception);
       if (GetMagickEncoderThreadSupport(magick_info) == MagickFalse)
@@ -1235,8 +1231,8 @@ MagickExport MagickBooleanType WriteImage(const ImageInfo *image_info,
               */
               if (GetMagickEncoderThreadSupport(magick_info) == MagickFalse)
                 LockSemaphoreInfo(magick_info->semaphore);
-              status=IsCoderAuthorized(magick_info->module,write_info->magick,
-                WritePolicyRights,exception);
+              status=IsCoderAuthorized(write_info->magick,WritePolicyRights,
+                exception);
               if (status != MagickFalse)
                 status=encoder(write_info,image,exception);
               if (GetMagickEncoderThreadSupport(magick_info) == MagickFalse)
index abddc26b0f02ecd5e5d675ec77dc47fd529e72fd..4471a7d87dca4a7cf6767efc1b069cf277813692 100644 (file)
@@ -557,9 +557,9 @@ MagickExport MagickBooleanType GlobExpression(const char *expression,
                   }
               }
             }
-            if (GetUTFCode(pattern) != 0)
-              pattern+=GetUTFOctets(pattern);
-          }
+          if (GetUTFCode(pattern) != 0)
+            pattern+=GetUTFOctets(pattern);
+        }
         break;
       }
       case '\\':