From: Simon Schubert <corecode@dragonflybsd.org>
Date: Tue, 25 Aug 2009 00:22:04 +0000 (+0200)
Subject: dma: don't set it->mailf too early
X-Git-Tag: v2.4.0~126^2
X-Git-Url: https://gitweb.dragonflybsd.org/dragonfly.git/commitdiff_plain/122842a9604b367cf0449900f26c6c452fe6d4d6

dma: don't set it->mailf too early

We may not set it->mailf in newspoolf(), or aquirespool() will not
(re-)open the mail file, leading to concurrent seeks.

Reported-by: Daniel Roethlisberger
DragonFly-Bug: <http://bugs.dragonflybsd.org/issue1462>
---

diff --git a/libexec/dma/spool.c b/libexec/dma/spool.c
index 9a74620220..33e972d508 100644
--- a/libexec/dma/spool.c
+++ b/libexec/dma/spool.c
@@ -101,7 +101,6 @@ newspoolf(struct queue *queue, const char *sender)
 	hdrlen = ftello(queue->mailf);
 
 	LIST_FOREACH(it, &queue->queue, next) {
-		it->mailf = queue->mailf;
 		it->hdrlen = hdrlen;
 	}