[Raw Msg Headers][Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

return_receipt() bug in mailbox.c



In transports/mailbox/mailbox.c in the function return_receipt, around 
line 3775, there is a code like this:

if (efp) {
   initialize boundarystr;
}

if (efp) {
   use boundarystr in some way;
} else {
   use boundarystr in another way;
}

As a result, if there is no efp, boundarystr is not initialized and 
contains junk.  Which leads to syntactically wrong delivery reports.

Presense of "return-receipt" form would probably be a quick fix (did not 
verify it yet) but otherwise, the first "if (efp)" apparently needs to 
be eliminated, and initialization be unconditional.

Eugene

OpenPGP digital signature