The following issue has been RESOLVED. 
====================================================================== 
http://dbmail.org/mantis/view.php?id=325 
====================================================================== 
Reported By:                piranha
Assigned To:                aaron
====================================================================== 
Project:                    DBMail
Issue ID:                   325
Category:                   PIPE delivery (dbmail-smtp)
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     resolved
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             17-Apr-06 18:27 CEST
Last Modified:              23-Apr-06 23:27 CEST
====================================================================== 
Summary:                    Broken pipe delivery for off-site addresses
Description: 
Delivery to off-site address, f.e.:
dbmail-users -x [EMAIL PROTECTED] -t [EMAIL PROTECTED]
is done by pipe which is broken. 

Line 104 in forward.c:
snprintf(command, command_len, "%s -f %s %s", sendmail, from,
Command line for sendmail is unquoted and needed to be:
snprintf(command, command_len, "%s -f \"%s\" \"%s\"", sendmail, from,

Because delivery of message with From such "My Name <[EMAIL PROTECTED]>" is
impossible in command line like:
sendmail -f My Name <[EMAIL PROTECTED]> [EMAIL PROTECTED]

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0000323 pipe to sendmail is opened incorrect
====================================================================== 

---------------------------------------------------------------------- 
 aaron - 17-Apr-06 21:43  
---------------------------------------------------------------------- 
I have a fix for this in my tree, but I haven't fully tested it yet. 

---------------------------------------------------------------------- 
 aaron - 18-Apr-06 08:11  
---------------------------------------------------------------------- 
Ok, let's try this patch on for size. It builds, but haven't tested if it
fully works! 

---------------------------------------------------------------------- 
 paul - 18-Apr-06 22:58  
---------------------------------------------------------------------- 
I've applied your fix Aaron since it passes the current test suites. It
does require more specific testing. 

---------------------------------------------------------------------- 
 aaron - 23-Apr-06 23:27  
---------------------------------------------------------------------- 
I changed what appears to be an off-by-one error. We need to write a
testcase for forwards. Resolving the bug since the patch is in and no
complaints have come back. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
17-Apr-06 18:27 piranha        New Issue                                    
17-Apr-06 21:43 aaron          Relationship added       related to 0000323  
17-Apr-06 21:43 aaron          Note Added: 0001083                          
18-Apr-06 08:11 aaron          File Added: dbmail-2.1-2006-04-17.diff           
        
18-Apr-06 08:11 aaron          Note Added: 0001086                          
18-Apr-06 22:58 paul           Note Added: 0001091                          
18-Apr-06 22:58 paul           Status                   new => confirmed    
23-Apr-06 23:27 aaron          Status                   confirmed => resolved
23-Apr-06 23:27 aaron          Resolution               open => fixed       
23-Apr-06 23:27 aaron          Assigned To               => aaron           
23-Apr-06 23:27 aaron          Note Added: 0001095                          
======================================================================

Reply via email to