From d1a17029a3eff704fd00c0f1f1d219869944fcc9 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Thu, 07 Nov 2013 08:46:25 -0500
Subject: [PATCH] Better handling of fopen() errors

---
 program/steps/mail/get.inc |   21 +++++++++++----------
 1 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/program/steps/mail/get.inc b/program/steps/mail/get.inc
index e0c4e29..ae48307 100644
--- a/program/steps/mail/get.inc
+++ b/program/steps/mail/get.inc
@@ -84,17 +84,18 @@
 
     // render thumbnail image if not done yet
     if (!is_file($cache_file)) {
-      $fp = fopen(($orig_name = $cache_basename . '.orig.' . $ext), 'w');
-      $MESSAGE->get_part_content($part->mime_id, $fp);
-      fclose($fp);
+      if ($fp = fopen(($orig_name = $cache_basename . '.orig.' . $ext), 'w')) {
+        $MESSAGE->get_part_content($part->mime_id, $fp);
+        fclose($fp);
 
-      $image = new rcube_image($orig_name);
-      if ($imgtype = $image->resize($thumbnail_size, $cache_file, true)) {
-        $mimetype = 'image/' . $imgtype;
-        unlink($orig_name);
-      }
-      else {
-        rename($orig_name, $cache_file);
+        $image = new rcube_image($orig_name);
+        if ($imgtype = $image->resize($thumbnail_size, $cache_file, true)) {
+          $mimetype = 'image/' . $imgtype;
+          unlink($orig_name);
+        }
+        else {
+          rename($orig_name, $cache_file);
+        }
       }
     }
 

--
Gitblit v1.9.1