iliaa           Mon Sep  8 18:36:59 2003 EDT

  Modified files:              
    /php-src/win32      sendmail.c 
  Log:
  Fixed bug #25333 (Possible body corruption & crash in win32 mail()).
  
  
Index: php-src/win32/sendmail.c
diff -u php-src/win32/sendmail.c:1.57 php-src/win32/sendmail.c:1.58
--- php-src/win32/sendmail.c:1.57       Mon Aug 11 14:03:24 2003
+++ php-src/win32/sendmail.c    Mon Sep  8 18:36:59 2003
@@ -17,7 +17,7 @@
  *
  */
 
-/* $Id: sendmail.c,v 1.57 2003/08/11 18:03:24 iliaa Exp $ */
+/* $Id: sendmail.c,v 1.58 2003/09/08 22:36:59 iliaa Exp $ */
 
 #include "php.h"                               /*php specific */
 #include <stdio.h>
@@ -385,7 +385,7 @@
 int SendText(char *RPath, char *Subject, char *mailTo, char *mailCc, char *mailBcc, 
char *data, 
                         char *headers, char *headers_lc, char **error_message)
 {
-       int res, i;
+       int res;
        char *p;
        char *tempMailTo, *token, *pos1, *pos2;
        char *server_response = NULL;
@@ -619,35 +619,30 @@
         * uses ZVAL as it's parameters */
        data_cln = php_str_to_str(data, strlen(data), PHP_WIN32_MAIL_DOT_PATTERN, 
sizeof(PHP_WIN32_MAIL_DOT_PATTERN) - 1,
                                        PHP_WIN32_MAIL_DOT_REPLACE, 
sizeof(PHP_WIN32_MAIL_DOT_REPLACE) - 1, &data_cln_len);
+       if (!data_cln) {
+               data_cln = estrdup("");
+               data_cln_len = 1;               
+       }
 
        /* send message contents in 1024 chunks */
-       if (data_cln_len <= 1024) {
-               if ((res = Post(data_cln)) != SUCCESS) {
-                       efree(data_cln);
-                       return (res);
-               }
-       } else {
-               int parts = (int) floor(data_cln_len / 1024);
+       {
+               char c, *e2, *e = data_cln + data_cln_len;
                p = data_cln;
 
-               for (i = 0; i < parts; i++) {
-                       strlcpy(Buffer, p, 1024);
-                       Buffer[1024] = '\0';
-                       p += 1024;
-send_chunk:
-                       /* send chunk */
-                       if ((res = Post(Buffer)) != SUCCESS) {
+               while (e - p > 1024) {
+                       e2 = p + 1024;
+                       c = *e2;
+                       *e2 = '\0';
+                       if ((res = Post(p)) != SUCCESS) {
                                efree(data_cln);
-                               return (res);
+                               return(res);
                        }
+                       *e2 = c;
+                       p = e2;
                }
-
-               if ((parts * 1024) < data_cln_len) {
-                       i = data_cln_len - (parts * 1024);
-                       strlcpy(Buffer, p, i);
-                       Buffer[i] = '\0';
-                       parts++;
-                       goto send_chunk;
+               if ((res = Post(p)) != SUCCESS) {
+                       efree(data_cln);
+                       return(res);
                }
        }
 

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to