Mailing List Archive: [Bug 401] DomainKeys dk_transport_write_message doesn't unlink an existing -K spool file

[Bug 401] DomainKeys dk_transport_write_message doesn't unlink an existing -K spool file

Oct 30, 2006, 4:05 AM

Post #1 of 4 (1047 views)

http://www.exim.org/bugzilla/show_bug.cgi?id=401

------- Additional Comments From bugzilla.exim.simon@arlott.org 2006-10-30 12:05 -------
Created an attachment (id=24)
--> (http://www.exim.org/bugzilla/attachment.cgi?id=24&action=view)
Unlink any existing -K spool file before trying to create it

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 401] DomainKeys dk_transport_write_message doesn't unlink an existing -K spool file [ In reply to ]

bugzilla-daemon at sesame

Oct 30, 2006, 10:27 AM

Post #2 of 4 (1021 views)

Permalink

[Bug 401] DomainKeys dk_transport_write_message doesn't unlink an existing -K spool file [ In reply to ]

bugzilla-daemon at sesame

Oct 30, 2006, 10:31 AM

Post #3 of 4 (1036 views)

Permalink

http://www.exim.org/bugzilla/show_bug.cgi?id=401

tom@duncanthrax.net changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 401] DomainKeys dk_transport_write_message doesn't unlink an existing -K spool file [ In reply to ]

bugzilla-daemon at sesame

Oct 30, 2006, 2:04 PM

Post #4 of 4 (1025 views)

Permalink

http://www.exim.org/bugzilla/show_bug.cgi?id=401

tom@duncanthrax.net changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED

------- Additional Comments From tom@duncanthrax.net 2006-10-30 22:04 -------
Did it in another way:

1) Create the -K file with an additional pid prefix to completely
rule out concurrency problems.
2) Change O_EXCL to O_TRUNC on file open to enable silent overwrite
of eventually existing file.
3) Use linux-style sendfile() call to send the -K file (when availablen,
and when TLS is not used). This should give a slight I/O performance
improvement.

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##