ID:               28976
 Comment by:       datarecovery at 163 dot com
 Reported By:      kdaniel at aosepc dot com
 Status:           Open
 Bug Type:         Mail related
 Operating System: Windows\Apache
 PHP Version:      4.3.6
 New Comment:

<A href="http://user1.7host.com/ycxxw";>上海盘夫数据恢复中心</A>
<A href="http://pafu.51.net/index.htm ">   上海盘夫数据恢复</A>
<A href="http://pafu.51.net";>   上海盘夫数据恢复</A>
<A href="http://pafu.w4.dns2008.cn/";>   上海盘夫数据恢复</A>
<A href="http://pafu.w4.dns2008.cn/case.htm ">   上海盘夫数据恢复</A>
<A href="http://pafu.w4.dns2008.cn/support.htm ">  
上海盘夫数据恢复</A>
<A href="http://pafu.w4.dns2008.cn/case.htm ">   上海盘夫数据恢复</A>
<A href="http://www.aimite.com/index.htm ">    数据恢复研究</A>
<A href="http://www.aimite.com/link.htm ">    数据恢复</A>
<A href="http://www.belinking.com/index.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n1.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n1-2.htm ">  上海硬盘数据恢复</A>
<A href="http://www.belinking.com/n1-3.htm ">  上海数据库恢复</A>
<A href="http://www.belinking.com/n1-4.htm ">  数据恢复</A>
<A href="http://hebu.myrice.com/index1.htm ">  数据恢复 </A>
<A href="http://hebu.myrice.com/luich.htm ">  数据恢复 </A>
<A href="http://hebu.myrice.com/price.htm ">  数据恢复 </A>
<A href="http://hebu.myrice.com/ts.htm ">  数据恢复 </A>
<A href="http://hebu.myrice.com/ts1.htm ">  数据恢复 </A>
<A href="http://hebu.myrice.com/ts2.htm ">  数据恢复 </A>
<A href="http://www.belinking.com/n1-5.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n1-6.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n2.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n3.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n4.htm ">  上海数据恢复</A>
<A href="http://www.belinking.com/n5.htm ">  上海数据恢复联系</A>
<A href="http://www.belinking.com ">  上海数据恢复</A>
<A href="http://hebu.51.net/doc.htm ">  数据恢复技术文档</A>
<A href="http://hebu.51.net/case.htm ">  数据恢复成功案例</A>
<A href="http://hebu.51.net/price.htm ">  数据恢复报价</A>
<A href="http://hebu.51.net/doc0.htm ">  硬盘内部结构图数据恢复</A>
<A href="http://hebu.51.net/doc1.htm ">  硬盘故障诊断与数据恢复</A>
<A href="http://hebu.51.net/doc2.htm ">  数据恢复</A>
<A href="http://hebu.51.net/doc3.htm "> 
数据恢复硬盘常见的一些故障处理</A>
<A href="http://hebu.51.net/doc4.htm ">  西部数据的工厂(上)</A>
<A href="http://hebu.51.net/lianxi.htm ">  数据恢复联系</A>
<A href="http://pafu.w4.dns2008.cn/intro.htm ">上海盘夫数据恢复</A>
<A href="http://pafu.w4.dns2008.cn/contact.htm ">上海盘夫数据恢复</A>
<A href="http://hebu.51.net/doc5.htm ">西部数据的工厂(下)</A>
<A href="http://hebu.51.net/doc6.htm ">硬盘马达技术剖析</A>
<A href="http://hebu.51.net/raid.htm ">磁盘阵列数据恢复</A>
<A href="http://hebu.51.net/doc7.htm ">硬盘数据保护技术</A>
<A href="http://hebu.51.net/hdd.htm ">硬盘数据恢复</A>
<A href="http://www.aimite.com/case.htm ">数据恢复成功案例</A>
<A href="http://www.aimite.com/contact.htm ">数据恢复联系我们</A>
<A href="http://www.aimite.com/map.htm ">数据恢复地图</A>
<A href="http://hebu.51.net/doc8.htm
">硬盘数据存储及其相关系统的原理</A>
<A href="http://hebu.51.net/doc9.htm ">数据恢复</A>
<A href="http://hebu.51.net/doc10.htm ">走进数据恢复</A>
<A href="http://hebu.51.net/doc11.htm
">一些盘夫开盘恢复数据的实践图片</A>
<A href="http://hebu.51.net/index.htm";>盘夫数据恢复</A>
<A href="http://hebu.myrice.com/case.htm ">数据恢复 </A>
<A href="http://hebu.myrice.com/contact.htm ">数据恢复 </A>
<A href="http://hebu.myrice.com/eqe.htm";>数据恢复 </A>
<A href="http://hebu.myrice.com/index.htm ">数据恢复</A>
<A href="http://hebu.myrice.com/ts3.htm ">数据恢复</A>
<A href="http://hebu.myrice.com/ts4.htm "> 数据恢复</A>
<A href="http://hebu.myrice.com/tse.htm "> 数据恢复</A>
<A href="http://recycle.myrice.com ">  数据恢复 </A>


Previous Comments:
------------------------------------------------------------------------

[2004-11-11 04:38:29] jpablo at dld dot net

The same happens with 4.3.7

------------------------------------------------------------------------

[2004-11-04 16:53:59] Charlesk at netgaintechnology dot com

It seems to me that this was thought of but never implemented.  In
sendmail.c:

        if (headers) {
                char *pos = NULL;
                size_t i;

                /* Use PCRE to trim the header into the right format */
                if (NULL == (headers = php_win32_mail_trim_header(headers
TSRMLS_CC))) {
                        *error = W32_SM_PCRE_ERROR;
                        return FAILURE;
                }

                /* Create a lowercased header for all the searches so we're 
finally
case
                 * insensitive when searching for a pattern. */
                if (NULL == (headers_lc = estrdup(headers))) {
                        efree(headers);
                        *error = OUT_OF_MEMORY;
                        return FAILURE;
                }
                for (i = 0; i < strlen(headers_lc); i++) {
                        headers_lc[i] = tolower(headers_lc[i]);
                }
        }
 
        /* Fall back to sendmail_from php.ini setting */
        if (mailRPath && *mailRPath) {
                RPath = estrdup(mailRPath);
        }
        else if (INI_STR("sendmail_from")) {
                RPath = estrdup(INI_STR("sendmail_from"));
        } else {
                if (headers) {
                        efree(headers);
                        efree(headers_lc);
                }
                *error = W32_SM_SENDMAIL_FROM_NOT_SET;
                return FAILURE;
        }


I would think that in the if (headers) section there should be some
code to set the mailRPath if the From header is found in headers.  That
way when it falls into the if (mailRPath && *mailRPath) section it will
appropriately assign the from to the return path.  

Can this be fixed soon?

Thanks
Charles

------------------------------------------------------------------------

[2004-08-24 19:53:10] dv at josheli dot com

http://marc.theaimsgroup.com/?l=php-dev&m=109286883222906&w=2

------------------------------------------------------------------------

[2004-08-23 19:19:30] kdaniel at aosepc dot com

Yes it is.  The problem exists in sendmail.c

The code as follows:

/* Fall back to sendmail_from php.ini setting */
        if (mailRPath && *mailRPath) {
                RPath = estrdup(mailRPath);
        }
        else if (INI_STR("sendmail_from")) {
                RPath = estrdup(INI_STR("sendmail_from"));
        } else {
                if (headers) {
                        efree(headers);
                        efree(headers_lc);
                }
                *error = W32_SM_SENDMAIL_FROM_NOT_SET;
                return FAILURE;
        }

Should be changed to:

/* Fall back to sendmail_from php.ini setting */
        if (mailRPath && *mailRPath) {
                RPath = estrdup(mailRPath);
        }
        else if (INI_STR("sendmail_from")) {
                RPath = estrdup(INI_STR("sendmail_from"));
        } else {
                if (headers) {
                        efree(headers);
                        efree(headers_lc);
                } else {
                        *error = W32_SM_SENDMAIL_FROM_NOT_SET;
                        return FAILURE;
                }
        }


I have not tested this, it is just a guess.

Kevin

------------------------------------------------------------------------

[2004-08-23 19:02:33] joe at inexo dot com

It's not coming from php.ini.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/28976

-- 
Edit this bug report at http://bugs.php.net/?id=28976&edit=1

Reply via email to