r23779: Change from v2 or later to v3 or later.
[tprouty/samba.git] / source3 / printing / printing.c
index 67b9b9f22ab698fa102c56c5f3a7ec270a706f9c..de2e4150d227150d0e36eef9aa6f7bb53b6405b3 100644 (file)
@@ -7,7 +7,7 @@
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
@@ -25,6 +25,8 @@
 
 extern SIG_ATOMIC_T got_sig_term;
 extern SIG_ATOMIC_T reload_after_sighup;
+extern struct current_user current_user;
+extern userdom_struct current_user_info;
 
 /* Current printer interface */
 static BOOL remove_from_jobs_changed(const char* sharename, uint32 jobid);
@@ -43,13 +45,6 @@ static BOOL remove_from_jobs_changed(const char* sharename, uint32 jobid);
    jobids are assigned when a job starts spooling. 
 */
 
-struct print_queue_update_context {
-       char* sharename;
-       enum printing_types printing_type;
-       char* lpqcommand;
-};
-
-
 static TDB_CONTEXT *rap_tdb;
 static uint16 next_rap_jobid;
 struct rap_jobid_key {
@@ -67,6 +62,7 @@ uint16 pjobid_to_rap(const char* sharename, uint32 jobid)
        uint16 rap_jobid;
        TDB_DATA data, key;
        struct rap_jobid_key jinfo;
+       uint8 buf[2];
 
        DEBUG(10,("pjobid_to_rap: called.\n"));
 
@@ -80,7 +76,7 @@ uint16 pjobid_to_rap(const char* sharename, uint32 jobid)
        ZERO_STRUCT( jinfo );
        fstrcpy( jinfo.sharename, sharename );
        jinfo.jobid = jobid;
-       key.dptr = (char*)&jinfo;
+       key.dptr = (uint8 *)&jinfo;
        key.dsize = sizeof(jinfo);
 
        data = tdb_fetch(rap_tdb, key);
@@ -96,7 +92,8 @@ uint16 pjobid_to_rap(const char* sharename, uint32 jobid)
        rap_jobid = ++next_rap_jobid;
        if (rap_jobid == 0)
                rap_jobid = ++next_rap_jobid;
-       data.dptr = (char *)&rap_jobid;
+       SSVAL(buf,0,rap_jobid);
+       data.dptr = buf;
        data.dsize = sizeof(rap_jobid);
        tdb_store(rap_tdb, key, data, TDB_REPLACE);
        tdb_store(rap_tdb, data, key, TDB_REPLACE);
@@ -109,13 +106,15 @@ uint16 pjobid_to_rap(const char* sharename, uint32 jobid)
 BOOL rap_to_pjobid(uint16 rap_jobid, fstring sharename, uint32 *pjobid)
 {
        TDB_DATA data, key;
+       uint8 buf[2];
 
        DEBUG(10,("rap_to_pjobid called.\n"));
 
        if (!rap_tdb)
                return False;
 
-       key.dptr = (char *)&rap_jobid;
+       SSVAL(buf,0,rap_jobid);
+       key.dptr = buf;
        key.dsize = sizeof(rap_jobid);
        data = tdb_fetch(rap_tdb, key);
        if ( data.dptr && data.dsize == sizeof(struct rap_jobid_key) ) 
@@ -140,6 +139,7 @@ static void rap_jobid_delete(const char* sharename, uint32 jobid)
        TDB_DATA key, data;
        uint16 rap_jobid;
        struct rap_jobid_key jinfo;
+       uint8 buf[2];
 
        DEBUG(10,("rap_jobid_delete: called.\n"));
 
@@ -149,7 +149,7 @@ static void rap_jobid_delete(const char* sharename, uint32 jobid)
        ZERO_STRUCT( jinfo );
        fstrcpy( jinfo.sharename, sharename );
        jinfo.jobid = jobid;
-       key.dptr = (char*)&jinfo;
+       key.dptr = (uint8 *)&jinfo;
        key.dsize = sizeof(jinfo);
 
        data = tdb_fetch(rap_tdb, key);
@@ -165,7 +165,8 @@ static void rap_jobid_delete(const char* sharename, uint32 jobid)
 
        rap_jobid = SVAL(data.dptr, 0);
        SAFE_FREE(data.dptr);
-       data.dptr = (char *)&rap_jobid;
+       SSVAL(buf,0,rap_jobid);
+       data.dptr = buf;
        data.dsize = sizeof(rap_jobid);
        tdb_delete(rap_tdb, key);
        tdb_delete(rap_tdb, data);
@@ -177,7 +178,7 @@ static int get_queue_status(const char* sharename, print_status_struct *);
  Initialise the printing backend. Called once at startup before the fork().
 ****************************************************************************/
 
-BOOL print_backend_init(void)
+BOOL print_backend_init(struct messaging_context *msg_ctx)
 {
        const char *sversion = "INFO/version";
        pstring printing_path;
@@ -198,7 +199,7 @@ BOOL print_backend_init(void)
                pdb = get_print_db_byname(lp_const_servicename(snum));
                if (!pdb)
                        continue;
-               if (tdb_lock_bystring(pdb->tdb, sversion, 0) == -1) {
+               if (tdb_lock_bystring(pdb->tdb, sversion) == -1) {
                        DEBUG(0,("print_backend_init: Failed to open printer %s database\n", lp_const_servicename(snum) ));
                        release_print_db(pdb);
                        return False;
@@ -214,7 +215,7 @@ BOOL print_backend_init(void)
        close_all_print_db(); /* Don't leave any open. */
 
        /* do NT print initialization... */
-       return nt_printing_init();
+       return nt_printing_init(msg_ctx);
 }
 
 /****************************************************************************
@@ -245,6 +246,12 @@ static struct printif *get_printer_fns_from_type( enum printing_types type )
        }
 #endif /* HAVE_CUPS */
 
+#ifdef HAVE_IPRINT
+       if ( type == PRINT_IPRINT ) {
+               printer_fns = &iprint_printif;
+       }
+#endif /* HAVE_IPRINT */
+
        printer_fns->type = type;
        
        return printer_fns;
@@ -252,7 +259,7 @@ static struct printif *get_printer_fns_from_type( enum printing_types type )
 
 static struct printif *get_printer_fns( int snum )
 {
-       return get_printer_fns_from_type( lp_printing(snum) );
+       return get_printer_fns_from_type( (enum printing_types)lp_printing(snum) );
 }
 
 
@@ -266,7 +273,7 @@ static TDB_DATA print_key(uint32 jobid)
        TDB_DATA ret;
 
        SIVAL(&j, 0, jobid);
-       ret.dptr = (void *)&j;
+       ret.dptr = (uint8 *)&j;
        ret.dsize = sizeof(j);
        return ret;
 }
@@ -275,7 +282,7 @@ static TDB_DATA print_key(uint32 jobid)
  unpack a pjob from a tdb buffer 
 ***********************************************************************/
  
-int unpack_pjob( char* buf, int buflen, struct printjob *pjob )
+int unpack_pjob( uint8 *buf, int buflen, struct printjob *pjob )
 {
        int     len = 0;
        int     used;
@@ -332,27 +339,38 @@ static struct printjob *print_job_find(const char *sharename, uint32 jobid)
        TDB_DATA                ret;
        struct tdb_print_db     *pdb = get_print_db_byname(sharename);
        
+       DEBUG(10,("print_job_find: looking up job %u for share %s\n",
+                       (unsigned int)jobid, sharename ));
 
-       if (!pdb)
+       if (!pdb) {
                return NULL;
+       }
 
        ret = tdb_fetch(pdb->tdb, print_key(jobid));
        release_print_db(pdb);
 
-       if (!ret.dptr)
+       if (!ret.dptr) {
+               DEBUG(10,("print_job_find: failed to find jobid %u.\n", (unsigned int)jobid ));
                return NULL;
+       }
        
-       if ( pjob.nt_devmode )
+       if ( pjob.nt_devmode ) {
                free_nt_devicemode( &pjob.nt_devmode );
+       }
                
        ZERO_STRUCT( pjob );
        
        if ( unpack_pjob( ret.dptr, ret.dsize, &pjob ) == -1 ) {
+               DEBUG(10,("print_job_find: failed to unpack jobid %u.\n", (unsigned int)jobid ));
                SAFE_FREE(ret.dptr);
                return NULL;
        }
        
-       SAFE_FREE(ret.dptr);    
+       SAFE_FREE(ret.dptr);
+
+       DEBUG(10,("print_job_find: returning system job %d for jobid %u.\n",
+                       (int)pjob.sysjob, (unsigned int)jobid ));
+
        return &pjob;
 }
 
@@ -400,8 +418,10 @@ uint32 sysjob_to_jobid(int unix_jobid)
                if (!lp_print_ok(snum))
                        continue;
                pdb = get_print_db_byname(lp_const_servicename(snum));
-               if (pdb)
-                       tdb_traverse(pdb->tdb, unixjob_traverse_fn, &unix_jobid);
+               if (!pdb) {
+                       continue;
+               }
+               tdb_traverse(pdb->tdb, unixjob_traverse_fn, &unix_jobid);
                release_print_db(pdb);
                if (sysjob_to_jobid_value != (uint32)-1)
                        return sysjob_to_jobid_value;
@@ -498,7 +518,7 @@ static BOOL pjob_store(const char* sharename, uint32 jobid, struct printjob *pjo
        TDB_DATA                old_data, new_data;
        BOOL                    ret = False;
        struct tdb_print_db     *pdb = get_print_db_byname(sharename);
-       char                    *buf = NULL;
+       uint8                   *buf = NULL;
        int                     len, newlen, buflen;
        
 
@@ -534,15 +554,11 @@ static BOOL pjob_store(const char* sharename, uint32 jobid, struct printjob *pjo
                len += pack_devicemode(pjob->nt_devmode, buf+len, buflen-len);
        
                if (buflen != len) {
-                       char *tb;
-
-                       tb = (char *)SMB_REALLOC(buf, len);
-                       if (!tb) {
+                       buf = (uint8 *)SMB_REALLOC(buf, len);
+                       if (!buf) {
                                DEBUG(0,("pjob_store: failed to enlarge buffer!\n"));
                                goto done;
                        }
-                       else 
-                               buf = tb;
                        newlen = len;
                }
        } while ( buflen != len );
@@ -684,6 +700,8 @@ struct traverse_struct {
        int qcount, snum, maxcount, total_jobs;
        const char *sharename;
        time_t lpq_time;
+       const char *lprm_command;
+       struct printif *print_if;
 };
 
 /****************************************************************************
@@ -730,7 +748,7 @@ static int traverse_fn_delete(TDB_CONTEXT *t, TDB_DATA key, TDB_DATA data, void
                /* if a job is not spooled and the process doesn't
                    exist then kill it. This cleans up after smbd
                    deaths */
-               if (!process_exists(pjob.pid)) {
+               if (!process_exists_by_pid(pjob.pid)) {
                        DEBUG(10,("traverse_fn_delete: pjob %u deleted due to !process_exists (%u)\n",
                                                (unsigned int)jobid, (unsigned int)pjob.pid ));
                        pjob_delete(ts->sharename, jobid);
@@ -743,9 +761,38 @@ static int traverse_fn_delete(TDB_CONTEXT *t, TDB_DATA key, TDB_DATA data, void
        
        if ( pjob.smbjob ) {
                for (i=0;i<ts->qcount;i++) {
-                       uint32 curr_jobid = print_parse_jobid(ts->queue[i].fs_file);
-                       if (jobid == curr_jobid)
+                       uint32 curr_jobid;
+
+                       if ( pjob.status == LPQ_DELETED )
+                               continue;
+
+                       curr_jobid = print_parse_jobid(ts->queue[i].fs_file);
+
+                       if (jobid == curr_jobid) {
+
+                               /* try to clean up any jobs that need to be deleted */
+
+                               if ( pjob.status == LPQ_DELETING ) {
+                                       int result;
+
+                                       result = (*(ts->print_if->job_delete))( 
+                                               ts->sharename, ts->lprm_command, &pjob );
+
+                                       if ( result != 0 ) {
+                                               /* if we can't delete, then reset the job status */
+                                               pjob.status = LPQ_QUEUED;
+                                               pjob_store(ts->sharename, jobid, &pjob);
+                                       }
+                                       else {
+                                               /* if we deleted the job, the remove the tdb record */
+                                               pjob_delete(ts->sharename, jobid);
+                                               pjob.status = LPQ_DELETED;
+                                       }
+                                               
+                               }
+
                                break;
+                       }
                }
        }
        
@@ -772,9 +819,10 @@ static int traverse_fn_delete(TDB_CONTEXT *t, TDB_DATA key, TDB_DATA data, void
                return 0;
        }
 
-       /* Save the pjob attributes we will store. */
-       /* FIXME!!! This is the only place where queue->job 
+       /* Save the pjob attributes we will store. 
+          FIXME!!! This is the only place where queue->job 
           represents the SMB jobid      --jerry */
+
        ts->queue[i].job = jobid;               
        ts->queue[i].size = pjob.size;
        ts->queue[i].page_count = pjob.page_count;
@@ -793,10 +841,9 @@ static int traverse_fn_delete(TDB_CONTEXT *t, TDB_DATA key, TDB_DATA data, void
  Check if the print queue has been updated recently enough.
 ****************************************************************************/
 
-static void print_cache_flush(int snum)
+static void print_cache_flush(const char *sharename)
 {
        fstring key;
-       const char *sharename = lp_const_servicename(snum);
        struct tdb_print_db *pdb = get_print_db_byname(sharename);
 
        if (!pdb)
@@ -810,7 +857,7 @@ static void print_cache_flush(int snum)
  Check if someone already thinks they are doing the update.
 ****************************************************************************/
 
-static pid_t get_updating_pid(fstring sharename)
+static pid_t get_updating_pid(const char *sharename)
 {
        fstring keystr;
        TDB_DATA data, key;
@@ -820,8 +867,7 @@ static pid_t get_updating_pid(fstring sharename)
        if (!pdb)
                return (pid_t)-1;
        slprintf(keystr, sizeof(keystr)-1, "UPDATING/%s", sharename);
-       key.dptr = keystr;
-       key.dsize = strlen(keystr);
+       key = string_tdb_data(keystr);
 
        data = tdb_fetch(pdb->tdb, key);
        release_print_db(pdb);
@@ -833,7 +879,7 @@ static pid_t get_updating_pid(fstring sharename)
        updating_pid = IVAL(data.dptr, 0);
        SAFE_FREE(data.dptr);
 
-       if (process_exists(updating_pid))
+       if (process_exists_by_pid(updating_pid))
                return updating_pid;
 
        return (pid_t)-1;
@@ -858,8 +904,7 @@ static void set_updating_pid(const fstring sharename, BOOL updating)
                return;
 
        slprintf(keystr, sizeof(keystr)-1, "UPDATING/%s", sharename);
-       key.dptr = keystr;
-       key.dsize = strlen(keystr);
+       key = string_tdb_data(keystr);
        
        DEBUG(5, ("set_updating_pid: %s updating lpq cache for print share %s\n", 
                updating ? "" : "not ",
@@ -872,7 +917,7 @@ static void set_updating_pid(const fstring sharename, BOOL updating)
        }
        
        SIVAL( buffer, 0, updating_pid);
-       data.dptr = (void *)buffer;
+       data.dptr = buffer;
        data.dsize = 4;         /* we always assume this is a 4 byte value */
 
        tdb_store(pdb->tdb, key, data, TDB_REPLACE);    
@@ -903,11 +948,12 @@ static int printjob_comp(print_queue_struct *j1, print_queue_struct *j2)
 
 /****************************************************************************
  Store the sorted queue representation for later portmon retrieval.
+ Skip deleted jobs
 ****************************************************************************/
 
 static void store_queue_struct(struct tdb_print_db *pdb, struct traverse_struct *pts)
 {
-       TDB_DATA data, key;
+       TDB_DATA data;
        int max_reported_jobs = lp_max_reported_jobs(pts->snum);
        print_queue_struct *queue = pts->queue;
        size_t len;
@@ -916,13 +962,17 @@ static void store_queue_struct(struct tdb_print_db *pdb, struct traverse_struct
 
        if (max_reported_jobs && (max_reported_jobs < pts->qcount))
                pts->qcount = max_reported_jobs;
-       qcount = pts->qcount;
+       qcount = 0;
 
        /* Work out the size. */
        data.dsize = 0;
        data.dsize += tdb_pack(NULL, 0, "d", qcount);
 
        for (i = 0; i < pts->qcount; i++) {
+               if ( queue[i].status == LPQ_DELETED )
+                       continue;
+
+               qcount++;
                data.dsize += tdb_pack(NULL, 0, "ddddddff",
                                (uint32)queue[i].job,
                                (uint32)queue[i].size,
@@ -934,12 +984,15 @@ static void store_queue_struct(struct tdb_print_db *pdb, struct traverse_struct
                                queue[i].fs_file);
        }
 
-       if ((data.dptr = SMB_MALLOC(data.dsize)) == NULL)
+       if ((data.dptr = (uint8 *)SMB_MALLOC(data.dsize)) == NULL)
                return;
 
         len = 0;
        len += tdb_pack(data.dptr + len, data.dsize - len, "d", qcount);
        for (i = 0; i < pts->qcount; i++) {
+               if ( queue[i].status == LPQ_DELETED )
+                       continue;
+
                len += tdb_pack(data.dptr + len, data.dsize - len, "ddddddff",
                                (uint32)queue[i].job,
                                (uint32)queue[i].size,
@@ -951,22 +1004,19 @@ static void store_queue_struct(struct tdb_print_db *pdb, struct traverse_struct
                                queue[i].fs_file);
        }
 
-       key.dptr = "INFO/linear_queue_array";
-       key.dsize = strlen(key.dptr);
-       tdb_store(pdb->tdb, key, data, TDB_REPLACE);
+       tdb_store(pdb->tdb, string_tdb_data("INFO/linear_queue_array"), data,
+                 TDB_REPLACE);
        SAFE_FREE(data.dptr);
        return;
 }
 
 static TDB_DATA get_jobs_changed_data(struct tdb_print_db *pdb)
 {
-       TDB_DATA data, key;
+       TDB_DATA data;
 
-       key.dptr = "INFO/jobs_changed";
-       key.dsize = strlen(key.dptr);
        ZERO_STRUCT(data);
 
-       data = tdb_fetch(pdb->tdb, key);
+       data = tdb_fetch(pdb->tdb, string_tdb_data("INFO/jobs_changed"));
        if (data.dptr == NULL || data.dsize == 0 || (data.dsize % 4 != 0)) {
                SAFE_FREE(data.dptr);
                ZERO_STRUCT(data);
@@ -1020,6 +1070,7 @@ static BOOL print_cache_expired(const char *sharename, BOOL check_pending)
                || (time_now - last_qscan_time) >= lp_lpqcachetime() 
                || last_qscan_time > (time_now + MAX_CACHE_VALID_TIME)) 
        {
+               uint32 u;
                time_t msg_pending_time;
 
                DEBUG(4, ("print_cache_expired: cache expired for queue %s " 
@@ -1035,8 +1086,8 @@ static BOOL print_cache_expired(const char *sharename, BOOL check_pending)
                snprintf(key, sizeof(key), "MSG_PENDING/%s", sharename);
 
                if ( check_pending 
-                       && tdb_fetch_uint32( pdb->tdb, key, &msg_pending_time ) 
-                       && msg_pending_time > 0
+                       && tdb_fetch_uint32( pdb->tdb, key, &u ) 
+                       && (msg_pending_time=u) > 0
                        && msg_pending_time <= time_now 
                        && (time_now - msg_pending_time) < 60 ) 
                {
@@ -1059,7 +1110,7 @@ done:
 
 static void print_queue_update_internal( const char *sharename, 
                                          struct printif *current_printif,
-                                         char *lpq_command )
+                                         char *lpq_command, char *lprm_command )
 {
        int i, qcount;
        print_queue_struct *queue = NULL;
@@ -1071,14 +1122,13 @@ static void print_queue_update_internal( const char *sharename,
        TDB_DATA jcdata;
        fstring keystr, cachestr;
        struct tdb_print_db *pdb = get_print_db_byname(sharename);
-       
-       DEBUG(5,("print_queue_update_internal: printer = %s, type = %d, lpq command = [%s]\n",
-               sharename, current_printif->type, lpq_command));
 
-       if ( !print_cache_expired(sharename, False) ) {
-               DEBUG(5,("print_queue_update_internal: print cache for %s is still ok\n", sharename));
+       if (!pdb) {
                return;
        }
+       
+       DEBUG(5,("print_queue_update_internal: printer = %s, type = %d, lpq command = [%s]\n",
+               sharename, current_printif->type, lpq_command));
 
        /*
         * Update the cache time FIRST ! Stops others even
@@ -1138,8 +1188,14 @@ static void print_queue_update_internal( const char *sharename,
                }
 
                pjob->sysjob = queue[i].job;
-               pjob->status = queue[i].status;
+
+               /* don't reset the status on jobs to be deleted */
+
+               if ( pjob->status != LPQ_DELETING )
+                       pjob->status = queue[i].status;
+
                pjob_store(sharename, jobid, pjob);
+
                check_job_changed(sharename, jcdata, jobid);
        }
 
@@ -1153,6 +1209,8 @@ static void print_queue_update_internal( const char *sharename,
        tstruct.total_jobs = 0;
        tstruct.lpq_time = time(NULL);
        tstruct.sharename = sharename;
+       tstruct.lprm_command = lprm_command;
+       tstruct.print_if = current_printif;
 
        tdb_traverse(pdb->tdb, traverse_fn_delete, (void *)&tstruct);
 
@@ -1173,11 +1231,10 @@ static void print_queue_update_internal( const char *sharename,
 
        /* store the new queue status structure */
        slprintf(keystr, sizeof(keystr)-1, "STATUS/%s", sharename);
-       key.dptr = keystr;
-       key.dsize = strlen(keystr);
+       key = string_tdb_data(keystr);
 
        status.qcount = qcount;
-       data.dptr = (void *)&status;
+       data.dptr = (uint8 *)&status;
        data.dsize = sizeof(status);
        tdb_store(pdb->tdb, key, data, TDB_REPLACE);    
 
@@ -1211,20 +1268,24 @@ static void print_queue_update_internal( const char *sharename,
  smbd processes maytry to update the lpq cache concurrently).
 ****************************************************************************/
 
-static void print_queue_update_with_lock(int snum)
+static void print_queue_update_with_lock( const char *sharename, 
+                                          struct printif *current_printif,
+                                          char *lpq_command, char *lprm_command )
 {
-       fstring sharename, keystr;
-       pstring lpq_command;
+       fstring keystr;
        struct tdb_print_db *pdb;
-       struct printif *current_printif = get_printer_fns( snum );
-
-       fstrcpy(sharename, lp_const_servicename(snum));
 
        DEBUG(5,("print_queue_update_with_lock: printer share = %s\n", sharename));
        pdb = get_print_db_byname(sharename);
        if (!pdb)
                return;
 
+       if ( !print_cache_expired(sharename, False) ) {
+               DEBUG(5,("print_queue_update_with_lock: print cache for %s is still ok\n", sharename));
+               release_print_db(pdb);
+               return;
+       }
+       
        /*
         * Check to see if someone else is doing this update.
         * This is essentially a mutex on the update.
@@ -1239,7 +1300,7 @@ static void print_queue_update_with_lock(int snum)
 
        slprintf(keystr, sizeof(keystr) - 1, "LOCK/%s", sharename);
        /* Only wait 10 seconds for this. */
-       if (tdb_lock_bystring(pdb->tdb, keystr, 10) == -1) {
+       if (tdb_lock_bystring_with_timeout(pdb->tdb, keystr, 10) == -1) {
                DEBUG(0,("print_queue_update_with_lock: Failed to lock printer %s database\n", sharename));
                release_print_db(pdb);
                return;
@@ -1275,14 +1336,9 @@ static void print_queue_update_with_lock(int snum)
        tdb_unlock_bystring(pdb->tdb, keystr);
 
        /* do the main work now */
-       /* have to substitute any variables here since 
-           print_queue_get_internal() will not */
        
-       pstrcpy( lpq_command, lp_lpqcommand(snum) );
-       pstring_sub( lpq_command, "%p", PRINTERNAME(snum) );
-       standard_sub_snum( snum, lpq_command, sizeof(lpq_command) );
-       
-       print_queue_update_internal( sharename, current_printif, lpq_command );
+       print_queue_update_internal( sharename, current_printif, 
+               lpq_command, lprm_command );
        
        /* Delete our pid from the db. */
        set_updating_pid(sharename, False);
@@ -1292,29 +1348,31 @@ static void print_queue_update_with_lock(int snum)
 /****************************************************************************
 this is the receive function of the background lpq updater
 ****************************************************************************/
-static void print_queue_receive(int msg_type, pid_t src, void *buf, size_t msglen)
+static void print_queue_receive(struct messaging_context *msg,
+                               void *private_data,
+                               uint32_t msg_type,
+                               struct server_id server_id,
+                               DATA_BLOB *data)
 {
-       struct print_queue_update_context ctx;
        fstring sharename;
-       pstring lpqcommand;
+       pstring lpqcommand, lprmcommand;
+       int printing_type;
        size_t len;
 
-       len = tdb_unpack( buf, msglen, "fdP",
+       len = tdb_unpack( (uint8 *)data->data, data->length, "fdPP",
                sharename,
-               &ctx.printing_type,
-               lpqcommand );
+               &printing_type,
+               lpqcommand,
+               lprmcommand );
 
        if ( len == -1 ) {
                DEBUG(0,("print_queue_receive: Got invalid print queue update message\n"));
                return;
        }
 
-       ctx.sharename = sharename;
-       ctx.lpqcommand = lpqcommand;
-
-       print_queue_update_internal(ctx.sharename, 
-               get_printer_fns_from_type(ctx.printing_type),
-               ctx.lpqcommand );
+       print_queue_update_with_lock(sharename, 
+               get_printer_fns_from_type((enum printing_types)printing_type),
+               lpqcommand, lprmcommand );
 
        return;
 }
@@ -1338,14 +1396,15 @@ void start_background_queue(void)
                /* Child. */
                DEBUG(5,("start_background_queue: background LPQ thread started\n"));
 
-               claim_connection( NULL, "smbd lpq backend", 0, False, 
+               claim_connection( NULL, "smbd lpq backend",
                        FLAG_MSG_GENERAL|FLAG_MSG_SMBD|FLAG_MSG_PRINT_GENERAL);
 
                if (!locking_init(0)) {
                        exit(1);
                }
 
-               message_register(MSG_PRINTER_UPDATE, print_queue_receive);
+               messaging_register(smbd_messaging_context(), NULL,
+                                  MSG_PRINTER_UPDATE, print_queue_receive);
                
                DEBUG(5,("start_background_queue: background LPQ thread waiting for messages\n"));
                while (1) {
@@ -1354,7 +1413,7 @@ void start_background_queue(void)
                        /* check for some essential signals first */
                        
                         if (got_sig_term) {
-                                exit_server("Caught TERM signal");
+                                exit_server_cleanly(NULL);
                         }
 
                         if (reload_after_sighup) {
@@ -1367,11 +1426,12 @@ void start_background_queue(void)
                        /* now check for messages */
                        
                        DEBUG(10,("start_background_queue: background LPQ thread got a message\n"));
-                       message_dispatch();
+                       message_dispatch(smbd_messaging_context());
 
                        /* process any pending print change notify messages */
 
-                       print_notify_send_messages(0);
+                       print_notify_send_messages(smbd_messaging_context(),
+                                                  0);
                }
        }
 }
@@ -1379,64 +1439,91 @@ void start_background_queue(void)
 /****************************************************************************
 update the internal database from the system print queue for a queue
 ****************************************************************************/
-static void print_queue_update(int snum)
+
+static void print_queue_update(int snum, BOOL force)
 {
-       struct print_queue_update_context ctx;
        fstring key;
        fstring sharename;
-       pstring lpqcommand;
-       char *buffer = NULL;
+       pstring lpqcommand, lprmcommand;
+       uint8 *buffer = NULL;
        size_t len = 0;
        size_t newlen;
        struct tdb_print_db *pdb;
+       int type;
+       struct printif *current_printif;
+
+       fstrcpy( sharename, lp_const_servicename(snum));
 
+       /* don't strip out characters like '$' from the printername */
+       
+       pstrcpy( lpqcommand, lp_lpqcommand(snum));
+       string_sub2( lpqcommand, "%p", PRINTERNAME(snum), sizeof(lpqcommand), 
+                    False, False, False );
+       standard_sub_advanced(lp_servicename(snum),
+                             current_user_info.unix_name, "",
+                             current_user.ut.gid,
+                             get_current_username(),
+                             current_user_info.domain,
+                             lpqcommand, sizeof(lpqcommand) );
+       
+       pstrcpy( lprmcommand, lp_lprmcommand(snum));
+       string_sub2( lprmcommand, "%p", PRINTERNAME(snum), sizeof(lprmcommand), 
+                    False, False, False );
+       standard_sub_advanced(lp_servicename(snum),
+                             current_user_info.unix_name, "",
+                             current_user.ut.gid,
+                             get_current_username(),
+                             current_user_info.domain,
+                             lprmcommand, sizeof(lprmcommand) );
+       
        /* 
         * Make sure that the background queue process exists.  
         * Otherwise just do the update ourselves 
         */
        
-       if ( background_lpq_updater_pid == -1 ) {
-               print_queue_update_with_lock( snum );
+       if ( force || background_lpq_updater_pid == -1 ) {
+               DEBUG(4,("print_queue_update: updating queue [%s] myself\n", sharename));
+               current_printif = get_printer_fns( snum );
+               print_queue_update_with_lock( sharename, current_printif, lpqcommand, lprmcommand );
+
                return;
        }
 
-       fstrcpy( sharename, lp_const_servicename(snum));
-
-       ctx.printing_type = lp_printing(snum);
-
-       pstrcpy( lpqcommand, lp_lpqcommand(snum));
-       pstring_sub( lpqcommand, "%p", PRINTERNAME(snum) );
-       standard_sub_snum( snum, lpqcommand, sizeof(lpqcommand) );
-
-       ctx.sharename = SMB_STRDUP( sharename );
-       ctx.lpqcommand = SMB_STRDUP( lpqcommand );
-
+       type = lp_printing(snum);
+       
        /* get the length */
 
-       len = tdb_pack( buffer, len, "fdP",
-               ctx.sharename,
-               ctx.printing_type,
-               ctx.lpqcommand );
+       len = tdb_pack( NULL, 0, "fdPP",
+               sharename,
+               type,
+               lpqcommand, 
+               lprmcommand );
 
-       buffer = SMB_XMALLOC_ARRAY( char, len );
+       buffer = SMB_XMALLOC_ARRAY( uint8, len );
 
        /* now pack the buffer */
-       newlen = tdb_pack( buffer, len, "fdP",
-               ctx.sharename,
-               ctx.printing_type,
-               ctx.lpqcommand );
+       newlen = tdb_pack( buffer, len, "fdPP",
+               sharename,
+               type,
+               lpqcommand,
+               lprmcommand );
 
        SMB_ASSERT( newlen == len );
 
        DEBUG(10,("print_queue_update: Sending message -> printer = %s, "
-               "type = %d, lpq command = [%s]\n",
-               ctx.sharename, ctx.printing_type, ctx.lpqcommand ));
+               "type = %d, lpq command = [%s] lprm command = [%s]\n", 
+               sharename, type, lpqcommand, lprmcommand ));
 
        /* here we set a msg pending record for other smbd processes 
           to throttle the number of duplicate print_queue_update msgs
           sent.  */
 
        pdb = get_print_db_byname(sharename);
+       if (!pdb) {
+               SAFE_FREE(buffer);
+               return;
+       }
+
        snprintf(key, sizeof(key), "MSG_PENDING/%s", sharename);
 
        if ( !tdb_store_uint32( pdb->tdb, key, time(NULL) ) ) {
@@ -1450,13 +1537,10 @@ static void print_queue_update(int snum)
 
        /* finally send the message */
        
-       become_root();
-       message_send_pid(background_lpq_updater_pid,
-                MSG_PRINTER_UPDATE, buffer, len, False);
-       unbecome_root();
+       messaging_send_buf(smbd_messaging_context(),
+                          pid_to_procid(background_lpq_updater_pid),
+                          MSG_PRINTER_UPDATE, (uint8 *)buffer, len);
 
-       SAFE_FREE( ctx.sharename );
-       SAFE_FREE( ctx.lpqcommand );
        SAFE_FREE( buffer );
 
        return;
@@ -1502,7 +1586,7 @@ BOOL print_notify_register_pid(int snum)
                tdb = pdb->tdb;
        }
 
-       if (tdb_lock_bystring(tdb, NOTIFY_PID_LIST_KEY, 10) == -1) {
+       if (tdb_lock_bystring_with_timeout(tdb, NOTIFY_PID_LIST_KEY, 10) == -1) {
                DEBUG(0,("print_notify_register_pid: Failed to lock printer %s\n",
                                        printername));
                if (pdb)
@@ -1524,7 +1608,7 @@ BOOL print_notify_register_pid(int snum)
 
        if (i == data.dsize) {
                /* We weren't in the list. Realloc. */
-               data.dptr = SMB_REALLOC(data.dptr, data.dsize + 8);
+               data.dptr = (uint8 *)SMB_REALLOC(data.dptr, data.dsize + 8);
                if (!data.dptr) {
                        DEBUG(0,("print_notify_register_pid: Relloc fail for printer %s\n",
                                                printername));
@@ -1592,7 +1676,7 @@ BOOL print_notify_deregister_pid(int snum)
                tdb = pdb->tdb;
        }
 
-       if (tdb_lock_bystring(tdb, NOTIFY_PID_LIST_KEY, 10) == -1) {
+       if (tdb_lock_bystring_with_timeout(tdb, NOTIFY_PID_LIST_KEY, 10) == -1) {
                DEBUG(0,("print_notify_register_pid: Failed to lock \
 printer %s database\n", printername));
                if (pdb)
@@ -1709,7 +1793,7 @@ NT_DEVICEMODE *print_job_devmode(const char* sharename, uint32 jobid)
  Set the place in the queue for a job.
 ****************************************************************************/
 
-BOOL print_job_set_place(int snum, uint32 jobid, int place)
+BOOL print_job_set_place(const char *sharename, uint32 jobid, int place)
 {
        DEBUG(2,("print_job_set_place not implemented yet\n"));
        return False;
@@ -1719,9 +1803,8 @@ BOOL print_job_set_place(int snum, uint32 jobid, int place)
  Set the name of a job. Only possible for owner.
 ****************************************************************************/
 
-BOOL print_job_set_name(int snum, uint32 jobid, char *name)
+BOOL print_job_set_name(const char *sharename, uint32 jobid, char *name)
 {
-       const char* sharename = lp_const_servicename(snum);
        struct printjob *pjob;
 
        pjob = print_job_find(sharename, jobid);
@@ -1744,10 +1827,14 @@ static BOOL remove_from_jobs_changed(const char* sharename, uint32 jobid)
        BOOL ret = False;
        BOOL gotlock = False;
 
-       key.dptr = "INFO/jobs_changed";
-       key.dsize = strlen(key.dptr);
+       if (!pdb) {
+               return False;
+       }
+
        ZERO_STRUCT(data);
 
+       key = string_tdb_data("INFO/jobs_changed");
+
        if (tdb_chainlock_with_timeout(pdb->tdb, key, 5) == -1)
                goto out;
 
@@ -1798,8 +1885,6 @@ static BOOL print_job_delete1(int snum, uint32 jobid)
        int result = 0;
        struct printif *current_printif = get_printer_fns( snum );
 
-       pjob = print_job_find(sharename, jobid);
-
        if (!pjob)
                return False;
 
@@ -1811,7 +1896,9 @@ static BOOL print_job_delete1(int snum, uint32 jobid)
                return True;
 
        /* Hrm - we need to be able to cope with deleting a job before it
-          has reached the spooler. */
+          has reached the spooler.  Just mark it as LPQ_DELETING and 
+          let the print_queue_update() code rmeove the record */
+          
 
        if (pjob->sysjob == -1) {
                DEBUG(5, ("attempt to delete job %u not seen by lpr\n", (unsigned int)jobid));
@@ -1822,24 +1909,31 @@ static BOOL print_job_delete1(int snum, uint32 jobid)
        pjob->status = LPQ_DELETING;
        pjob_store(sharename, jobid, pjob);
 
-       if (pjob->spooled && pjob->sysjob != -1)
-               result = (*(current_printif->job_delete))(snum, pjob);
+       if (pjob->spooled && pjob->sysjob != -1) 
+       {
+               result = (*(current_printif->job_delete))(
+                       PRINTERNAME(snum),
+                       lp_lprmcommand(snum), 
+                       pjob);
 
-       /* Delete the tdb entry if the delete succeeded or the job hasn't
-          been spooled. */
+               /* Delete the tdb entry if the delete succeeded or the job hasn't
+                  been spooled. */
 
-       if (result == 0) {
-               struct tdb_print_db *pdb = get_print_db_byname(sharename);
-               int njobs = 1;
+               if (result == 0) {
+                       struct tdb_print_db *pdb = get_print_db_byname(sharename);
+                       int njobs = 1;
 
-               if (!pdb)
-                       return False;
-               pjob_delete(sharename, jobid);
-               /* Ensure we keep a rough count of the number of total jobs... */
-               tdb_change_int32_atomic(pdb->tdb, "INFO/total_jobs", &njobs, -1);
-               release_print_db(pdb);
+                       if (!pdb)
+                               return False;
+                       pjob_delete(sharename, jobid);
+                       /* Ensure we keep a rough count of the number of total jobs... */
+                       tdb_change_int32_atomic(pdb->tdb, "INFO/total_jobs", &njobs, -1);
+                       release_print_db(pdb);
+               }
        }
 
+       remove_from_jobs_changed( sharename, jobid );
+
        return (result == 0);
 }
 
@@ -1847,9 +1941,10 @@ static BOOL print_job_delete1(int snum, uint32 jobid)
  Return true if the current user owns the print job.
 ****************************************************************************/
 
-static BOOL is_owner(struct current_user *user, int snum, uint32 jobid)
+static BOOL is_owner(struct current_user *user, const char *servicename,
+                    uint32 jobid)
 {
-       struct printjob *pjob = print_job_find(lp_const_servicename(snum), jobid);
+       struct printjob *pjob = print_job_find(servicename, jobid);
        user_struct *vuser;
 
        if (!pjob || !user)
@@ -1858,7 +1953,7 @@ static BOOL is_owner(struct current_user *user, int snum, uint32 jobid)
        if ((vuser = get_valid_user_struct(user->vuid)) != NULL) {
                return strequal(pjob->user, vuser->user.smb_name);
        } else {
-               return strequal(pjob->user, uidtoname(user->uid));
+               return strequal(pjob->user, uidtoname(user->ut.uid));
        }
 }
 
@@ -1869,12 +1964,13 @@ static BOOL is_owner(struct current_user *user, int snum, uint32 jobid)
 BOOL print_job_delete(struct current_user *user, int snum, uint32 jobid, WERROR *errcode)
 {
        const char* sharename = lp_const_servicename( snum );
-       BOOL    owner, deleted;
+       struct printjob *pjob;
+       BOOL    owner;
        char    *fname;
 
        *errcode = WERR_OK;
                
-       owner = is_owner(user, snum, jobid);
+       owner = is_owner(user, lp_const_servicename(snum), jobid);
        
        /* Check access against security descriptor or whether the user
           owns their job. */
@@ -1888,7 +1984,7 @@ BOOL print_job_delete(struct current_user *user, int snum, uint32 jobid, WERROR
                sys_adminlog( LOG_ERR, 
                              "Permission denied-- user not allowed to delete, \
 pause, or resume print job. User name: %s. Printer name: %s.",
-                             uidtoname(user->uid), PRINTERNAME(snum) );
+                             uidtoname(user->ut.uid), PRINTERNAME(snum) );
                /* END_ADMIN_LOG */
 
                return False;
@@ -1919,13 +2015,13 @@ pause, or resume print job. User name: %s. Printer name: %s.",
        /* force update the database and say the delete failed if the
            job still exists */
 
-       print_queue_update(snum);
+       print_queue_update(snum, True);
        
-       deleted = !print_job_exists(sharename, jobid);
-       if ( !deleted )
+       pjob = print_job_find(sharename, jobid);
+       if ( pjob && (pjob->status != LPQ_DELETING) )
                *errcode = WERR_ACCESS_DENIED;
 
-       return deleted;
+       return (pjob == NULL );
 }
 
 /****************************************************************************
@@ -1941,13 +2037,19 @@ BOOL print_job_pause(struct current_user *user, int snum, uint32 jobid, WERROR *
 
        pjob = print_job_find(sharename, jobid);
        
-       if (!pjob || !user) 
+       if (!pjob || !user) {
+               DEBUG(10, ("print_job_pause: no pjob or user for jobid %u\n",
+                       (unsigned int)jobid ));
                return False;
+       }
 
-       if (!pjob->spooled || pjob->sysjob == -1) 
+       if (!pjob->spooled || pjob->sysjob == -1) {
+               DEBUG(10, ("print_job_pause: not spooled or bad sysjob = %d for jobid %u\n",
+                       (int)pjob->sysjob, (unsigned int)jobid ));
                return False;
+       }
 
-       if (!is_owner(user, snum, jobid) &&
+       if (!is_owner(user, lp_const_servicename(snum), jobid) &&
            !print_access_check(user, snum, JOB_ACCESS_ADMINISTER)) {
                DEBUG(3, ("pause denied by security descriptor\n"));
 
@@ -1955,7 +2057,7 @@ BOOL print_job_pause(struct current_user *user, int snum, uint32 jobid, WERROR *
                sys_adminlog( LOG_ERR, 
                        "Permission denied-- user not allowed to delete, \
 pause, or resume print job. User name: %s. Printer name: %s.",
-                               uidtoname(user->uid), PRINTERNAME(snum) );
+                               uidtoname(user->ut.uid), PRINTERNAME(snum) );
                /* END_ADMIN_LOG */
 
                *errcode = WERR_ACCESS_DENIED;
@@ -1971,7 +2073,7 @@ pause, or resume print job. User name: %s. Printer name: %s.",
        }
 
        /* force update the database */
-       print_cache_flush(snum);
+       print_cache_flush(lp_const_servicename(snum));
 
        /* Send a printer notify message */
 
@@ -1995,13 +2097,19 @@ BOOL print_job_resume(struct current_user *user, int snum, uint32 jobid, WERROR
 
        pjob = print_job_find(sharename, jobid);
        
-       if (!pjob || !user)
+       if (!pjob || !user) {
+               DEBUG(10, ("print_job_resume: no pjob or user for jobid %u\n",
+                       (unsigned int)jobid ));
                return False;
+       }
 
-       if (!pjob->spooled || pjob->sysjob == -1)
+       if (!pjob->spooled || pjob->sysjob == -1) {
+               DEBUG(10, ("print_job_resume: not spooled or bad sysjob = %d for jobid %u\n",
+                       (int)pjob->sysjob, (unsigned int)jobid ));
                return False;
+       }
 
-       if (!is_owner(user, snum, jobid) &&
+       if (!is_owner(user, lp_const_servicename(snum), jobid) &&
            !print_access_check(user, snum, JOB_ACCESS_ADMINISTER)) {
                DEBUG(3, ("resume denied by security descriptor\n"));
                *errcode = WERR_ACCESS_DENIED;
@@ -2010,7 +2118,7 @@ BOOL print_job_resume(struct current_user *user, int snum, uint32 jobid, WERROR
                sys_adminlog( LOG_ERR, 
                         "Permission denied-- user not allowed to delete, \
 pause, or resume print job. User name: %s. Printer name: %s.",
-                       uidtoname(user->uid), PRINTERNAME(snum) );
+                       uidtoname(user->ut.uid), PRINTERNAME(snum) );
                /* END_ADMIN_LOG */
                return False;
        }
@@ -2023,7 +2131,7 @@ pause, or resume print job. User name: %s. Printer name: %s.",
        }
 
        /* force update the database */
-       print_cache_flush(snum);
+       print_cache_flush(lp_const_servicename(snum));
 
        /* Send a printer notify message */
 
@@ -2036,7 +2144,7 @@ pause, or resume print job. User name: %s. Printer name: %s.",
  Write to a print file.
 ****************************************************************************/
 
-int print_job_write(int snum, uint32 jobid, const char *buf, int size)
+ssize_t print_job_write(int snum, uint32 jobid, const char *buf, SMB_OFF_T pos, size_t size)
 {
        const char* sharename = lp_const_servicename(snum);
        int return_code;
@@ -2050,7 +2158,8 @@ int print_job_write(int snum, uint32 jobid, const char *buf, int size)
        if (pjob->pid != sys_getpid())
                return -1;
 
-       return_code = write(pjob->fd, buf, size);
+       return_code = write_data_at_offset(pjob->fd, buf, size, pos);
+
        if (return_code>0) {
                pjob->size += size;
                pjob_store(sharename, jobid, pjob);
@@ -2065,19 +2174,20 @@ int print_job_write(int snum, uint32 jobid, const char *buf, int size)
 static int get_queue_status(const char* sharename, print_status_struct *status)
 {
        fstring keystr;
-       TDB_DATA data, key;
+       TDB_DATA data;
        struct tdb_print_db *pdb = get_print_db_byname(sharename);
        int len;
 
+       if (status) {
+               ZERO_STRUCTP(status);
+       }
+
        if (!pdb)
                return 0;
 
        if (status) {
-               ZERO_STRUCTP(status);
-               slprintf(keystr, sizeof(keystr)-1, "STATUS/%s", sharename);
-               key.dptr = keystr;
-               key.dsize = strlen(keystr);
-               data = tdb_fetch(pdb->tdb, key);
+               fstr_sprintf(keystr, "STATUS/%s", sharename);
+               data = tdb_fetch(pdb->tdb, string_tdb_data(keystr));
                if (data.dptr) {
                        if (data.dsize == sizeof(print_status_struct))
                                /* this memcpy is ok since the status struct was 
@@ -2100,10 +2210,12 @@ int print_queue_length(int snum, print_status_struct *pstatus)
        const char* sharename = lp_const_servicename( snum );
        print_status_struct status;
        int len;
+
+       ZERO_STRUCT( status );
  
        /* make sure the database is up to date */
        if (print_cache_expired(lp_const_servicename(snum), True))
-               print_queue_update(snum);
+               print_queue_update(snum, False);
  
        /* also fetch the queue status */
        memset(&status, 0, sizeof(status));
@@ -2128,7 +2240,7 @@ static BOOL allocate_print_jobid(struct tdb_print_db *pdb, int snum, const char
 
        for (i = 0; i < 3; i++) {
                /* Lock the database - only wait 20 seconds. */
-               if (tdb_lock_bystring(pdb->tdb, "INFO/nextjob", 20) == -1) {
+               if (tdb_lock_bystring_with_timeout(pdb->tdb, "INFO/nextjob", 20) == -1) {
                        DEBUG(0,("allocate_print_jobid: failed to lock printing database %s\n", sharename));
                        return False;
                }
@@ -2187,18 +2299,17 @@ static BOOL allocate_print_jobid(struct tdb_print_db *pdb, int snum, const char
 
 static BOOL add_to_jobs_changed(struct tdb_print_db *pdb, uint32 jobid)
 {
-       TDB_DATA data, key;
+       TDB_DATA data;
        uint32 store_jobid;
 
-       key.dptr = "INFO/jobs_changed";
-       key.dsize = strlen(key.dptr);
        SIVAL(&store_jobid, 0, jobid);
-       data.dptr = (char *)&store_jobid;
+       data.dptr = (uint8 *)&store_jobid;
        data.dsize = 4;
 
        DEBUG(10,("add_to_jobs_changed: Added jobid %u\n", (unsigned int)jobid ));
 
-       return (tdb_append(pdb->tdb, key, data) == 0);
+       return (tdb_append(pdb->tdb, string_tdb_data("INFO/jobs_changed"),
+                          data) == 0);
 }
 
 /***************************************************************************
@@ -2226,7 +2337,7 @@ uint32 print_job_start(struct current_user *user, int snum, char *jobname, NT_DE
                return (uint32)-1;
        }
 
-       if (!print_time_access_check(snum)) {
+       if (!print_time_access_check(lp_servicename(snum))) {
                DEBUG(3, ("print_job_start: job start denied by time check\n"));
                release_print_db(pdb);
                return (uint32)-1;
@@ -2286,9 +2397,13 @@ uint32 print_job_start(struct current_user *user, int snum, char *jobname, NT_DE
        fstrcpy(pjob.jobname, jobname);
 
        if ((vuser = get_valid_user_struct(user->vuid)) != NULL) {
-               fstrcpy(pjob.user, vuser->user.smb_name);
+               fstrcpy(pjob.user, lp_printjob_username(snum));
+               standard_sub_basic(vuser->user.smb_name, vuser->user.domain, 
+                                  pjob.user, sizeof(pjob.user)-1);
+               /* ensure NULL termination */ 
+               pjob.user[sizeof(pjob.user)-1] = '\0'; 
        } else {
-               fstrcpy(pjob.user, uidtoname(user->uid));
+               fstrcpy(pjob.user, uidtoname(user->ut.uid));
        }
 
        fstrcpy(pjob.queuename, lp_const_servicename(snum));
@@ -2359,7 +2474,7 @@ void print_job_endpage(int snum, uint32 jobid)
  error.
 ****************************************************************************/
 
-BOOL print_job_end(int snum, uint32 jobid, BOOL normal_close)
+BOOL print_job_end(int snum, uint32 jobid, enum file_close_type close_type)
 {
        const char* sharename = lp_const_servicename(snum);
        struct printjob *pjob;
@@ -2375,7 +2490,8 @@ BOOL print_job_end(int snum, uint32 jobid, BOOL normal_close)
        if (pjob->spooled || pjob->pid != sys_getpid())
                return False;
 
-       if (normal_close && (sys_fstat(pjob->fd, &sbuf) == 0)) {
+       if ((close_type == NORMAL_CLOSE || close_type == SHUTDOWN_CLOSE) &&
+                               (sys_fstat(pjob->fd, &sbuf) == 0)) {
                pjob->size = sbuf.st_size;
                close(pjob->fd);
                pjob->fd = -1;
@@ -2418,7 +2534,7 @@ BOOL print_job_end(int snum, uint32 jobid, BOOL normal_close)
        
        /* make sure the database is up to date */
        if (print_cache_expired(lp_const_servicename(snum), True))
-               print_queue_update(snum);
+               print_queue_update(snum, False);
        
        return True;
 
@@ -2437,7 +2553,7 @@ fail:
 
 static BOOL get_stored_queue_info(struct tdb_print_db *pdb, int snum, int *pcount, print_queue_struct **ppqueue)
 {
-       TDB_DATA data, key, cgdata;
+       TDB_DATA data, cgdata;
        print_queue_struct *queue = NULL;
        uint32 qcount = 0;
        uint32 extra_count = 0;
@@ -2450,27 +2566,22 @@ static BOOL get_stored_queue_info(struct tdb_print_db *pdb, int snum, int *pcoun
 
        /* make sure the database is up to date */
        if (print_cache_expired(lp_const_servicename(snum), True))
-               print_queue_update(snum);
+               print_queue_update(snum, False);
  
        *pcount = 0;
        *ppqueue = NULL;
 
        ZERO_STRUCT(data);
        ZERO_STRUCT(cgdata);
-       key.dptr = "INFO/linear_queue_array";
-       key.dsize = strlen(key.dptr);
 
        /* Get the stored queue data. */
-       data = tdb_fetch(pdb->tdb, key);
+       data = tdb_fetch(pdb->tdb, string_tdb_data("INFO/linear_queue_array"));
        
        if (data.dptr && data.dsize >= sizeof(qcount))
                len += tdb_unpack(data.dptr + len, data.dsize - len, "d", &qcount);
                
        /* Get the changed jobs list. */
-       key.dptr = "INFO/jobs_changed";
-       key.dsize = strlen(key.dptr);
-
-       cgdata = tdb_fetch(pdb->tdb, key);
+       cgdata = tdb_fetch(pdb->tdb, string_tdb_data("INFO/jobs_changed"));
        if (cgdata.dptr != NULL && (cgdata.dsize % 4 == 0))
                extra_count = cgdata.dsize/4;
 
@@ -2571,7 +2682,7 @@ int print_queue_status(int snum,
        /* make sure the database is up to date */
 
        if (print_cache_expired(lp_const_servicename(snum), True))
-               print_queue_update(snum);
+               print_queue_update(snum, False);
 
        /* return if we are done */
        if ( !ppqueue || !status )
@@ -2591,8 +2702,8 @@ int print_queue_status(int snum,
 
        ZERO_STRUCTP(status);
        slprintf(keystr, sizeof(keystr)-1, "STATUS/%s", sharename);
-       key.dptr = keystr;
-       key.dsize = strlen(keystr);
+       key = string_tdb_data(keystr);
+
        data = tdb_fetch(pdb->tdb, key);
        if (data.dptr) {
                if (data.dsize == sizeof(*status)) {
@@ -2630,16 +2741,21 @@ BOOL print_queue_pause(struct current_user *user, int snum, WERROR *errcode)
                *errcode = WERR_ACCESS_DENIED;
                return False;
        }
+       
 
+       become_root();
+               
        ret = (*(current_printif->queue_pause))(snum);
 
+       unbecome_root();
+               
        if (ret != 0) {
                *errcode = WERR_INVALID_PARAM;
                return False;
        }
 
        /* force update the database */
-       print_cache_flush(snum);
+       print_cache_flush(lp_const_servicename(snum));
 
        /* Send a printer notify message */
 
@@ -2661,9 +2777,13 @@ BOOL print_queue_resume(struct current_user *user, int snum, WERROR *errcode)
                *errcode = WERR_ACCESS_DENIED;
                return False;
        }
-
+       
+       become_root();
+               
        ret = (*(current_printif->queue_resume))(snum);
 
+       unbecome_root();
+               
        if (ret != 0) {
                *errcode = WERR_INVALID_PARAM;
                return False;
@@ -2671,7 +2791,7 @@ BOOL print_queue_resume(struct current_user *user, int snum, WERROR *errcode)
 
        /* make sure the database is up to date */
        if (print_cache_expired(lp_const_servicename(snum), True))
-               print_queue_update(snum);
+               print_queue_update(snum, True);
 
        /* Send a printer notify message */
 
@@ -2692,18 +2812,27 @@ BOOL print_queue_purge(struct current_user *user, int snum, WERROR *errcode)
        BOOL can_job_admin;
 
        /* Force and update so the count is accurate (i.e. not a cached count) */
-       print_queue_update(snum);
+       print_queue_update(snum, True);
        
        can_job_admin = print_access_check(user, snum, JOB_ACCESS_ADMINISTER);
        njobs = print_queue_status(snum, &queue, &status);
+       
+       if ( can_job_admin )
+               become_root();
 
        for (i=0;i<njobs;i++) {
-               BOOL owner = is_owner(user, snum, queue[i].job);
+               BOOL owner = is_owner(user, lp_const_servicename(snum), queue[i].job);
 
                if (owner || can_job_admin) {
                        print_job_delete1(snum, queue[i].job);
                }
        }
+       
+       if ( can_job_admin )
+               unbecome_root();
+
+       /* update the cache */
+       print_queue_update( snum, True );
 
        SAFE_FREE(queue);