cool, thanks that patch is for 5.4 (since the line nums changed from 5.3-5.4,
On Thu, Aug 4, 2011 at 5:19 PM, Pierre Joye <pierre....@gmail.com> wrote: > I have now attached the patch to the issue: > https://bugs.php.net/bug.php?id=30688 > > On Thu, Aug 4, 2011 at 6:07 PM, Keloran <ava...@gmail.com> wrote: > > patch attached > > > > On Thu, Aug 4, 2011 at 4:39 PM, Pierre Joye <pierre....@gmail.com> > wrote: > >> > >> hi, > >> > >> Can you attach this patch to the bug please? > >> > >> Thanks for your work! > >> > >> On Thu, Aug 4, 2011 at 5:30 PM, Keloran <ava...@gmail.com> wrote: > >> > Index: ext/imap/php_imap.c > >> > =================================================================== > >> > --- ext/imap/php_imap.c (revision 314217) > >> > +++ ext/imap/php_imap.c (working copy) > >> > @@ -4016,7 +4016,27 @@ > >> > if (!INI_STR("sendmail_path")) { > >> > return 0; > >> > } > >> > - sendmail = popen(INI_STR("sendmail_path"), "w"); > >> > + > >> > + /** Used to make return-path work **/ > >> > + char *sendmail_path = INI_STR("sendmail_path"); > >> > + char *appended_sendmail_path = NULL; > >> > + > >> > + /** Return Path or not **/ > >> > + if (rpath && rpath[0]) { > >> > + appended_sendmail_path = emalloc( > >> > + strlen(sendmail_path) + 3 + strlen(rpath) + 1); > >> > + strcpy(appended_sendmail_path, sendmail_path); > >> > + strcat(appended_sendmail_path, " -f"); > >> > + strcat(appended_sendmail_path, rpath); > >> > + sendmail_path = appended_sendmail_path; > >> > + } > >> > + > >> > + /** open the sendmail pointer **/ > >> > + sendmail = popen(sendmail_path, "w"); /* New Code */ > >> > + > >> > + if (appended_sendmail_path) > >> > + efree(appended_sendmail_path); > >> > + > >> > if (sendmail) { > >> > if (rpath && rpath[0]) fprintf(sendmail, "From: %s\n", > >> > rpath); > >> > fprintf(sendmail, "To: %s\n", to); > >> > > >> > ================ > >> > > >> > hopefully that will work to fix the rpath option, and finally close > this > >> > bug http://bugs.php.net/bug.php?id=30688 > >> > > >> > >> > >> > >> -- > >> Pierre > >> > >> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org > > > > > > > > -- > Pierre > > @pierrejoye | http://blog.thepimp.net | http://www.libgd.org >
