[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.


OpenPGP digital signature