From e8d43bbfe4c05f813a014fd23b8b491ed2d8d4fa Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 15 Nov 2000 01:11:38 +0000 Subject: Tuyrn debug timestamps on by default. Add Tim's lpq race fix. Jeremy. (This used to be commit d43405bc47c95bf8c906035cba23bf95d252d13b) --- source3/printing/printing.c | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) (limited to 'source3/printing') diff --git a/source3/printing/printing.c b/source3/printing/printing.c index 8b17e8d5f2..2d98aa9974 100644 --- a/source3/printing/printing.c +++ b/source3/printing/printing.c @@ -271,11 +271,23 @@ static int traverse_fn_delete(TDB_CONTEXT *t, TDB_DATA key, TDB_DATA data, void if (jobid == qid) break; } + /* The job isn't in the system queue - we have to assume it has + completed, so delete the database entry. */ + if (i == ts->qcount) { - /* the job isn't in the system queue - we have to - assume it has completed, so delete the database - entry */ - tdb_delete(t, key); + time_t cur_t = time(NULL); + + /* A race can occur between the time a job is spooled and + when it appears in the lpq output. This happens when + the job is added to printing.tdb when another smbd + running print_queue_update() has completed a lpq and + is currently traversing the printing tdb and deleting jobs. + A workaround is to not delete the job if it has been + submitted less than lp_lpqcachetime() seconds ago. */ + + if ((cur_t - pjob.starttime) > lp_lpqcachetime()) { + tdb_delete(t, key); + } } return 0; -- cgit