Fix regression in resizing JPEG images with Imagick (#5376)

roundcube · Jul 29, 2016 · 2f2765a · 2f2765a
1 parent 672ca96
commit 2f2765a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -25,6 +25,7 @@ CHANGELOG Roundcube Webmail
 - Managesieve: Unhide advanced rule controls if there are inputs with errors
 - Managesieve: Display warning message when filter form contains errors
 - Enigma: Add possibility to configure gpg-agent binary location (enigma_pgp_agent)
+- Fix regression in resizing JPEG images with Imagick (#5376)
 
 RELEASE 1.2.1
 -------------

diff --git a/program/lib/Roundcube/rcube_image.php b/program/lib/Roundcube/rcube_image.php
@@ -167,9 +167,15 @@ public function resize($size, $filename = null, $browser_compat = false)
                         try {
                             $image = new Imagick($this->image_file);
 
-                            $image->setImageBackgroundColor('white');
-                            $image->setImageAlphaChannel(11);
-                            $image->mergeImageLayers(Imagick::LAYERMETHOD_FLATTEN);
+                            try {
+                                // it throws exception on formats not supporting these features
+                                $image->setImageBackgroundColor('white');
+                                $image->setImageAlphaChannel(11);
+                                $image->mergeImageLayers(Imagick::LAYERMETHOD_FLATTEN);
+                            }
+                            catch (Exception $e) {
+                                // ignore errors
+                            }
 
                             $image->setImageColorspace(Imagick::COLORSPACE_SRGB);
                             $image->setImageCompressionQuality(75);