Changeset 99f2b31 in github

Timestamp:

Dec 29, 2008 7:32:16 AM (4 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

a1733f0

Parents:

17060ee

Message:

#1485653: fix enless loop in rcmail_attach_emoticons() + fix attaching the same image a few times

File:

• program/steps/mail/sendmail.inc (modified) (2 diffs)

program/steps/mail/sendmail.inc

@@ -99 +99 @@
   
   $last_img_pos = 0;
-
   $searchstr = 'program/js/tiny_mce/plugins/emotions/img/';
+  $path_len = strlen(INSTALL_PATH . '/');
 
   // keep track of added images, so they're only added once
@@ -110 +110 @@
     $pos2 = strpos($body, '"', $pos);
     $body_pre = substr($body, 0, $pos);
+    $body_post = substr($body, $pos2);
+
     $image_name = substr($body,
                          $pos + strlen($searchstr),
                          $pos2 - ($pos + strlen($searchstr)));
+
     // sanitize image name so resulting attachment doesn't leave images dir
     $image_name = preg_replace('/[^a-zA-Z0-9_\.\-]/i','',$image_name);
-
-    $body_post = substr($body, $pos2);
+    $img_file = INSTALL_PATH . '/' . $searchstr . $image_name;
 
     if (! in_array($image_name, $included_images))
       {
       // add the image to the MIME message
-      $img_file = INSTALL_PATH . '/' . $searchstr . $image_name;
       if(! $mime_message->addHTMLImage($img_file, 'image/gif', '', true, $image_name))
         $OUTPUT->show_message("emoticonerror", 'error');
-
       array_push($included_images, $image_name);
       }
-
+    
     $body = $body_pre . $img_file . $body_post;
 
-    $last_img_pos = $pos2;
-    }
-   
+    $last_img_pos = $pos2 + $path_len;
+    }
+
   $mime_message->setHTMLBody($body);
 }