s3:printing: Fix size check in get_file_version()
[samba.git] / source3 / printing / nt_printing.c
index 7d1dd9e4d0cb35c092c445fe1a22e4194ffe69bc..241af37743ef9b1f0ecb59239743938bbf11f982 100644 (file)
@@ -29,6 +29,7 @@
 #include "../libcli/security/security.h"
 #include "passdb/machine_sid.h"
 #include "smbd/smbd.h"
+#include "smbd/globals.h"
 #include "auth.h"
 #include "messages.h"
 #include "rpc_server/spoolss/srv_spoolss_nt.h"
@@ -73,6 +74,152 @@ static const struct print_architecture_table_node archi_table[]= {
        {NULL,                   "",            -1 }
 };
 
+static bool print_driver_directories_init(void)
+{
+       int service, i;
+       char *driver_path;
+       bool ok;
+       TALLOC_CTX *mem_ctx = talloc_stackframe();
+       const char *dir_list[] = {
+               "W32X86/PCC",
+               "x64/PCC",
+               "color"
+       };
+
+       service = lp_servicenumber("print$");
+       if (service < 0) {
+               /* We don't have a print$ share */
+               DEBUG(5, ("No print$ share has been configured.\n"));
+               talloc_free(mem_ctx);
+               return true;
+       }
+
+       driver_path = lp_path(mem_ctx, service);
+       if (driver_path == NULL) {
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       ok = directory_create_or_exist(driver_path, 0755);
+       if (!ok) {
+               DEBUG(1, ("Failed to create printer driver directory %s\n",
+                         driver_path));
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       for (i = 0; archi_table[i].long_archi != NULL; i++) {
+               const char *arch_path;
+
+               arch_path = talloc_asprintf(mem_ctx,
+                                           "%s/%s",
+                                           driver_path,
+                                           archi_table[i].short_archi);
+               if (arch_path == NULL) {
+                       talloc_free(mem_ctx);
+                       return false;
+               }
+
+               ok = directory_create_or_exist(arch_path, 0755);
+               if (!ok) {
+                       DEBUG(1, ("Failed to create printer driver "
+                                 "architecture directory %s\n",
+                                 arch_path));
+                       talloc_free(mem_ctx);
+                       return false;
+               }
+       }
+
+       for (i = 0; i < ARRAY_SIZE(dir_list); i++) {
+               const char *path;
+
+               path = talloc_asprintf(mem_ctx,
+                                      "%s/%s",
+                                      driver_path,
+                                      dir_list[i]);
+               if (path == NULL) {
+                       talloc_free(mem_ctx);
+                       return false;
+               }
+
+               ok = directory_create_or_exist(path, 0755);
+               if (!ok) {
+                       DEBUG(1, ("Failed to create printer driver "
+                                 "architecture directory %s\n",
+                                 path));
+                       talloc_free(mem_ctx);
+                       return false;
+               }
+       }
+
+       driver_path = state_path("DriverStore");
+       if (driver_path == NULL) {
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       ok = directory_create_or_exist(driver_path, 0755);
+       if (!ok) {
+               DEBUG(1,("failed to create path %s\n", driver_path));
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       driver_path = state_path("DriverStore/FileRepository");
+       if (driver_path == NULL) {
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       ok = directory_create_or_exist(driver_path, 0755);
+       if (!ok) {
+               DEBUG(1,("failed to create path %s\n", driver_path));
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       driver_path = state_path("DriverStore/Temp");
+       if (driver_path == NULL) {
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       ok = directory_create_or_exist(driver_path, 0755);
+       if (!ok) {
+               DEBUG(1,("failed to create path %s\n", driver_path));
+               talloc_free(mem_ctx);
+               return false;
+       }
+
+       talloc_free(mem_ctx);
+       return true;
+}
+
+/****************************************************************************
+ Forward a MSG_PRINTER_DRVUPGRADE message from another smbd to the
+ background lpq updater.
+****************************************************************************/
+
+static void forward_drv_upgrade_printer_msg(struct messaging_context *msg,
+                               void *private_data,
+                               uint32_t msg_type,
+                               struct server_id server_id,
+                               DATA_BLOB *data)
+{
+       extern pid_t background_lpq_updater_pid;
+
+       if (background_lpq_updater_pid == -1) {
+               DEBUG(3,("no background lpq queue updater\n"));
+               return;
+       }
+
+       messaging_send_buf(msg,
+                       pid_to_procid(background_lpq_updater_pid),
+                       MSG_PRINTER_DRVUPGRADE,
+                       data->data,
+                       data->length);
+}
+
 /****************************************************************************
  Open the NT printing tdbs. Done once before fork().
 ****************************************************************************/
@@ -81,20 +228,20 @@ bool nt_printing_init(struct messaging_context *msg_ctx)
 {
        WERROR win_rc;
 
+       if (!print_driver_directories_init()) {
+               return false;
+       }
+
        if (!nt_printing_tdb_upgrade()) {
                return false;
        }
 
        /*
         * register callback to handle updating printers as new
-        * drivers are installed
+        * drivers are installed. Forwards to background lpq updater.
         */
        messaging_register(msg_ctx, NULL, MSG_PRINTER_DRVUPGRADE,
-                          do_drv_upgrade_printer);
-
-       /* of course, none of the message callbacks matter if you don't
-          tell messages.c that you interested in receiving PRINT_GENERAL
-          msgs.  This is done in serverid_register() */
+                       forward_drv_upgrade_printer_msg);
 
        if ( lp_security() == SEC_ADS ) {
                win_rc = check_published_printers(msg_ctx);
@@ -172,7 +319,7 @@ const char *get_short_archi(const char *long_archi)
  returns -1 on error, 1 on version info found, and 0 on no version info found.
 ****************************************************************************/
 
-static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32 *minor)
+static int get_file_version(files_struct *fsp, char *fname,uint32_t *major, uint32_t *minor)
 {
        int     i;
        char    *buf = NULL;
@@ -198,7 +345,7 @@ static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32
        }
 
        /* Skip OEM header (if any) and the DOS stub to start of Windows header */
-       if (SMB_VFS_LSEEK(fsp, SVAL(buf,DOS_HEADER_LFANEW_OFFSET), SEEK_SET) == (SMB_OFF_T)-1) {
+       if (SMB_VFS_LSEEK(fsp, SVAL(buf,DOS_HEADER_LFANEW_OFFSET), SEEK_SET) == (off_t)-1) {
                DEBUG(3,("get_file_version: File [%s] too short, errno = %d\n",
                                fname, errno));
                /* Assume this isn't an error... the file just looks sort of like a PE/NE file */
@@ -221,7 +368,7 @@ static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32
                /* Just skip over optional header to get to section table */
                if (SMB_VFS_LSEEK(fsp,
                                SVAL(buf,PE_HEADER_OPTIONAL_HEADER_SIZE)-(NE_HEADER_SIZE-PE_HEADER_SIZE),
-                               SEEK_CUR) == (SMB_OFF_T)-1) {
+                               SEEK_CUR) == (off_t)-1) {
                        DEBUG(3,("get_file_version: File [%s] Windows optional header too short, errno = %d\n",
                                fname, errno));
                        goto error_exit;
@@ -265,7 +412,7 @@ static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32
                                }
 
                                /* Seek to the start of the .rsrc section info */
-                               if (SMB_VFS_LSEEK(fsp, section_pos, SEEK_SET) == (SMB_OFF_T)-1) {
+                               if (SMB_VFS_LSEEK(fsp, section_pos, SEEK_SET) == (off_t)-1) {
                                        DEBUG(3,("get_file_version: PE file [%s] too short for section info, errno = %d\n",
                                                        fname, errno));
                                        goto error_exit;
@@ -311,7 +458,7 @@ static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32
                if (CVAL(buf,NE_HEADER_TARGET_OS_OFFSET) != NE_HEADER_TARGOS_WIN ) {
                        DEBUG(3,("get_file_version: NE file [%s] wrong target OS = 0x%x\n",
                                        fname, CVAL(buf,NE_HEADER_TARGET_OS_OFFSET)));
-                       /* At this point, we assume the file is in error. It still could be somthing
+                       /* At this point, we assume the file is in error. It still could be something
                         * else besides a NE file, but it unlikely at this point. */
                        goto error_exit;
                }
@@ -338,19 +485,31 @@ static int get_file_version(files_struct *fsp, char *fname,uint32 *major, uint32
                                /* Potential match data crosses buf boundry, move it to beginning
                                 * of buf, and fill the buf with as much as it will hold. */
                                if (i>byte_count-VS_VERSION_INFO_SIZE) {
-                                       int bc;
+                                       ssize_t amount_read;
+                                       ssize_t amount_unused = byte_count-i;
 
-                                       memcpy(buf, &buf[i], byte_count-i);
-                                       if ((bc = vfs_read_data(fsp, &buf[byte_count-i], VS_NE_BUF_SIZE-
-                                                                  (byte_count-i))) < 0) {
+                                       memmove(buf, &buf[i], amount_unused);
+                                       amount_read = vfs_read_data(fsp,
+                                               &buf[amount_unused],
+                                               VS_NE_BUF_SIZE- amount_unused);
+                                       if (amount_read < 0) {
 
                                                DEBUG(0,("get_file_version: NE file [%s] Read error, errno=%d\n",
                                                                 fname, errno));
                                                goto error_exit;
                                        }
 
-                                       byte_count = bc + (byte_count - i);
-                                       if (byte_count<VS_VERSION_INFO_SIZE) break;
+                                       if (amount_read + amount_unused <
+                                                       amount_read) {
+                                               /* Check for integer wrap. */
+                                               break;
+                                       }
+
+                                       byte_count = amount_read +
+                                                    amount_unused;
+                                       if (byte_count < VS_VERSION_INFO_SIZE) {
+                                               break;
+                                       }
 
                                        i = 0;
                                }
@@ -410,12 +569,12 @@ static int file_version_is_newer(connection_struct *conn, fstring new_file, fstr
 {
        bool use_version = true;
 
-       uint32 new_major;
-       uint32 new_minor;
+       uint32_t new_major;
+       uint32_t new_minor;
        time_t new_create_time;
 
-       uint32 old_major;
-       uint32 old_minor;
+       uint32_t old_major;
+       uint32_t old_minor;
        time_t old_create_time;
 
        struct smb_filename *smb_fname = NULL;
@@ -446,12 +605,14 @@ static int file_version_is_newer(connection_struct *conn, fstring new_file, fstr
                0,                                      /* create_options */
                FILE_ATTRIBUTE_NORMAL,                  /* file_attributes */
                INTERNAL_OPEN_ONLY,                     /* oplock_request */
+               NULL,                                   /* lease */
                0,                                      /* allocation_size */
                0,                                      /* private_flags */
                NULL,                                   /* sd */
                NULL,                                   /* ea_list */
                &fsp,                                   /* result */
-               NULL);                                  /* pinfo */
+               NULL,                                   /* pinfo */
+               NULL, NULL);                            /* create context */
 
        if (!NT_STATUS_IS_OK(status)) {
                /* Old file not found, so by definition new file is in fact newer */
@@ -499,12 +660,14 @@ static int file_version_is_newer(connection_struct *conn, fstring new_file, fstr
                0,                                      /* create_options */
                FILE_ATTRIBUTE_NORMAL,                  /* file_attributes */
                INTERNAL_OPEN_ONLY,                     /* oplock_request */
+               NULL,                                   /* lease */
                0,                                      /* allocation_size */
                0,                                      /* private_flags */
                NULL,                                   /* sd */
                NULL,                                   /* ea_list */
                &fsp,                                   /* result */
-               NULL);                                  /* pinfo */
+               NULL,                                   /* pinfo */
+               NULL, NULL);                            /* create context */
 
        if (!NT_STATUS_IS_OK(status)) {
                /* New file not found, this shouldn't occur if the caller did its job */
@@ -574,22 +737,24 @@ static int file_version_is_newer(connection_struct *conn, fstring new_file, fstr
 /****************************************************************************
 Determine the correct cVersion associated with an architecture and driver
 ****************************************************************************/
-static uint32 get_correct_cversion(struct auth_session_info *session_info,
+static uint32_t get_correct_cversion(struct auth_session_info *session_info,
                                   const char *architecture,
                                   const char *driverpath_in,
+                                  const char *driver_directory,
                                   WERROR *perr)
 {
        int cversion = -1;
        NTSTATUS          nt_status;
        struct smb_filename *smb_fname = NULL;
-       char *driverpath = NULL;
        files_struct      *fsp = NULL;
        connection_struct *conn = NULL;
-       char *oldcwd;
+       struct smb_filename *oldcwd_fname = NULL;
        char *printdollar = NULL;
+       char *printdollar_path = NULL;
+       char *working_dir = NULL;
        int printdollar_snum;
 
-       *perr = WERR_INVALID_PARAM;
+       *perr = WERR_INVALID_PARAMETER;
 
        /* If architecture is Windows 95/98/ME, the version is always 0. */
        if (strcmp(architecture, SPL_ARCH_WIN40) == 0) {
@@ -607,17 +772,42 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
 
        printdollar_snum = find_service(talloc_tos(), "print$", &printdollar);
        if (!printdollar) {
-               *perr = WERR_NOMEM;
+               *perr = WERR_NOT_ENOUGH_MEMORY;
                return -1;
        }
        if (printdollar_snum == -1) {
-               *perr = WERR_NO_SUCH_SHARE;
+               *perr = WERR_BAD_NET_NAME;
+               return -1;
+       }
+
+       printdollar_path = lp_path(talloc_tos(), printdollar_snum);
+       if (printdollar_path == NULL) {
+               *perr = WERR_NOT_ENOUGH_MEMORY;
                return -1;
        }
 
-       nt_status = create_conn_struct(talloc_tos(), &conn, printdollar_snum,
-                                      lp_pathname(printdollar_snum),
-                                      session_info, &oldcwd);
+       working_dir = talloc_asprintf(talloc_tos(),
+                                     "%s/%s",
+                                     printdollar_path,
+                                     architecture);
+       /*
+        * If the driver has been uploaded into a temorpary driver
+        * directory, switch to the driver directory.
+        */
+       if (driver_directory != NULL) {
+               working_dir = talloc_asprintf(talloc_tos(), "%s/%s/%s",
+                                             printdollar_path,
+                                             architecture,
+                                             driver_directory);
+       }
+
+       nt_status = create_conn_struct_cwd(talloc_tos(),
+                                          server_event_context(),
+                                          server_messaging_context(),
+                                          &conn,
+                                          printdollar_snum,
+                                          working_dir,
+                                          session_info, &oldcwd_fname);
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0,("get_correct_cversion: create_conn_struct "
                         "returned %s\n", nt_errstr(nt_status)));
@@ -638,18 +828,11 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
                goto error_free_conn;
        }
 
-       /* Open the driver file (Portable Executable format) and determine the
-        * deriver the cversion. */
-       driverpath = talloc_asprintf(talloc_tos(),
-                                       "%s/%s",
-                                       architecture,
-                                       driverpath_in);
-       if (!driverpath) {
-               *perr = WERR_NOMEM;
-               goto error_exit;
-       }
-
-       nt_status = driver_unix_convert(conn, driverpath, &smb_fname);
+       /*
+        * We switch to the directory where the driver files are located,
+        * so only work on the file names
+        */
+       nt_status = driver_unix_convert(conn, driverpath_in, &smb_fname);
        if (!NT_STATUS_IS_OK(nt_status)) {
                *perr = ntstatus_to_werror(nt_status);
                goto error_exit;
@@ -658,7 +841,7 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
        nt_status = vfs_file_exist(conn, smb_fname);
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(3,("get_correct_cversion: vfs_file_exist failed\n"));
-               *perr = WERR_BADFILE;
+               *perr = WERR_FILE_NOT_FOUND;
                goto error_exit;
        }
 
@@ -673,12 +856,14 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
                0,                                      /* create_options */
                FILE_ATTRIBUTE_NORMAL,                  /* file_attributes */
                INTERNAL_OPEN_ONLY,                     /* oplock_request */
+               NULL,                                   /* lease */
                0,                                      /* private_flags */
                0,                                      /* allocation_size */
                NULL,                                   /* sd */
                NULL,                                   /* ea_list */
                &fsp,                                   /* result */
-               NULL);                                  /* pinfo */
+               NULL,                                   /* pinfo */
+               NULL, NULL);                            /* create context */
 
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(3,("get_correct_cversion: Can't open file [%s], errno = "
@@ -686,19 +871,19 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
                *perr = WERR_ACCESS_DENIED;
                goto error_exit;
        } else {
-               uint32 major;
-               uint32 minor;
+               uint32_t major;
+               uint32_t minor;
                int    ret;
 
                ret = get_file_version(fsp, smb_fname->base_name, &major, &minor);
                if (ret == -1) {
-                       *perr = WERR_INVALID_PARAM;
+                       *perr = WERR_INVALID_PARAMETER;
                        goto error_exit;
                } else if (!ret) {
                        DEBUG(6,("get_correct_cversion: Version info not "
                                 "found [%s]\n",
                                 smb_fname_str_dbg(smb_fname)));
-                       *perr = WERR_INVALID_PARAM;
+                       *perr = WERR_INVALID_PARAMETER;
                        goto error_exit;
                }
 
@@ -740,11 +925,12 @@ static uint32 get_correct_cversion(struct auth_session_info *session_info,
                close_file(NULL, fsp, NORMAL_CLOSE);
        }
        if (conn != NULL) {
-               vfs_ChDir(conn, oldcwd);
+               vfs_ChDir(conn, oldcwd_fname);
+               TALLOC_FREE(oldcwd_fname);
                SMB_VFS_DISCONNECT(conn);
                conn_free(conn);
        }
-       if (!NT_STATUS_IS_OK(*perr)) {
+       if (!W_ERROR_IS_OK(*perr)) {
                cversion = -1;
        }
 
@@ -769,7 +955,9 @@ static WERROR clean_up_driver_struct_level(TALLOC_CTX *mem_ctx,
                                           const char **config_file,
                                           const char **help_file,
                                           struct spoolss_StringArray *dependent_files,
-                                          enum spoolss_DriverOSVersion *version)
+                                          enum spoolss_DriverOSVersion *version,
+                                          uint32_t flags,
+                                          const char **driver_directory)
 {
        const char *short_architecture;
        int i;
@@ -777,11 +965,48 @@ static WERROR clean_up_driver_struct_level(TALLOC_CTX *mem_ctx,
        char *_p;
 
        if (!*driver_path || !*data_file) {
-               return WERR_INVALID_PARAM;
+               return WERR_INVALID_PARAMETER;
        }
 
        if (!strequal(architecture, SPOOLSS_ARCHITECTURE_4_0) && !*config_file) {
-               return WERR_INVALID_PARAM;
+               return WERR_INVALID_PARAMETER;
+       }
+
+       if (flags & APD_COPY_FROM_DIRECTORY) {
+               char *path;
+               char *q;
+
+               /*
+                * driver_path is set to:
+                *
+                * \\PRINTSRV\print$\x64\{279245b0-a8bd-4431-bf6f-baee92ac15c0}\pscript5.dll
+                */
+               path = talloc_strdup(mem_ctx, *driver_path);
+               if (path == NULL) {
+                       return WERR_NOT_ENOUGH_MEMORY;
+               }
+
+               /* Remove pscript5.dll */
+               q = strrchr_m(path, '\\');
+               if (q == NULL) {
+                       return WERR_INVALID_PARAMETER;
+               }
+               *q = '\0';
+
+               /* Get \{279245b0-a8bd-4431-bf6f-baee92ac15c0} */
+               q = strrchr_m(path, '\\');
+               if (q == NULL) {
+                       return WERR_INVALID_PARAMETER;
+               }
+
+               /*
+                * Set driver_directory to:
+                *
+                * {279245b0-a8bd-4431-bf6f-baee92ac15c0}
+                *
+                * This is the directory where all the files have been uploaded
+                */
+               *driver_directory = q + 1;
        }
 
        /* clean up the driver name.
@@ -822,8 +1047,11 @@ static WERROR clean_up_driver_struct_level(TALLOC_CTX *mem_ctx,
         *      NT2K: cversion=3
         */
 
-       *version = get_correct_cversion(session_info, short_architecture,
-                                       *driver_path, &err);
+       *version = get_correct_cversion(session_info,
+                                       short_architecture,
+                                       *driver_path,
+                                       *driver_directory,
+                                       &err);
        if (*version == -1) {
                return err;
        }
@@ -836,7 +1064,9 @@ static WERROR clean_up_driver_struct_level(TALLOC_CTX *mem_ctx,
 
 WERROR clean_up_driver_struct(TALLOC_CTX *mem_ctx,
                              struct auth_session_info *session_info,
-                             struct spoolss_AddDriverInfoCtr *r)
+                             struct spoolss_AddDriverInfoCtr *r,
+                             uint32_t flags,
+                             const char **driver_directory)
 {
        switch (r->level) {
        case 3:
@@ -847,7 +1077,9 @@ WERROR clean_up_driver_struct(TALLOC_CTX *mem_ctx,
                                                    &r->info.info3->config_file,
                                                    &r->info.info3->help_file,
                                                    r->info.info3->dependent_files,
-                                                   &r->info.info3->version);
+                                                   &r->info.info3->version,
+                                                   flags,
+                                                   driver_directory);
        case 6:
                return clean_up_driver_struct_level(mem_ctx, session_info,
                                                    r->info.info6->architecture,
@@ -856,7 +1088,20 @@ WERROR clean_up_driver_struct(TALLOC_CTX *mem_ctx,
                                                    &r->info.info6->config_file,
                                                    &r->info.info6->help_file,
                                                    r->info.info6->dependent_files,
-                                                   &r->info.info6->version);
+                                                   &r->info.info6->version,
+                                                   flags,
+                                                   driver_directory);
+       case 8:
+               return clean_up_driver_struct_level(mem_ctx, session_info,
+                                                   r->info.info8->architecture,
+                                                   &r->info.info8->driver_path,
+                                                   &r->info.info8->data_file,
+                                                   &r->info.info8->config_file,
+                                                   &r->info.info8->help_file,
+                                                   r->info.info8->dependent_files,
+                                                   &r->info.info8->version,
+                                                   flags,
+                                                   driver_directory);
        default:
                return WERR_NOT_SUPPORTED;
        }
@@ -883,6 +1128,23 @@ static void convert_level_6_to_level3(struct spoolss_AddDriverInfo3 *dst,
        dst->dependent_files    = src->dependent_files;
 }
 
+static void convert_level_8_to_level3(struct spoolss_AddDriverInfo3 *dst,
+                                     const struct spoolss_AddDriverInfo8 *src)
+{
+       dst->version            = src->version;
+
+       dst->driver_name        = src->driver_name;
+       dst->architecture       = src->architecture;
+       dst->driver_path        = src->driver_path;
+       dst->data_file          = src->data_file;
+       dst->config_file        = src->config_file;
+       dst->help_file          = src->help_file;
+       dst->monitor_name       = src->monitor_name;
+       dst->default_datatype   = src->default_datatype;
+       dst->_ndr_size_dependent_files = src->_ndr_size_dependent_files;
+       dst->dependent_files    = src->dependent_files;
+}
+
 /****************************************************************************
 ****************************************************************************/
 
@@ -891,7 +1153,8 @@ static WERROR move_driver_file_to_download_area(TALLOC_CTX *mem_ctx,
                                                const char *driver_file,
                                                const char *short_architecture,
                                                uint32_t driver_version,
-                                               uint32_t version)
+                                               uint32_t version,
+                                               const char *driver_directory)
 {
        struct smb_filename *smb_fname_old = NULL;
        struct smb_filename *smb_fname_new = NULL;
@@ -900,29 +1163,41 @@ static WERROR move_driver_file_to_download_area(TALLOC_CTX *mem_ctx,
        NTSTATUS status;
        WERROR ret;
 
-       old_name = talloc_asprintf(mem_ctx, "%s/%s",
-                                  short_architecture, driver_file);
-       W_ERROR_HAVE_NO_MEMORY(old_name);
+       if (driver_directory != NULL) {
+               old_name = talloc_asprintf(mem_ctx,
+                                          "%s/%s/%s",
+                                          short_architecture,
+                                          driver_directory,
+                                          driver_file);
+       } else {
+               old_name = talloc_asprintf(mem_ctx,
+                                          "%s/%s",
+                                          short_architecture,
+                                          driver_file);
+       }
+       if (old_name == NULL) {
+               return WERR_NOT_ENOUGH_MEMORY;
+       }
 
        new_name = talloc_asprintf(mem_ctx, "%s/%d/%s",
                                   short_architecture, driver_version, driver_file);
        if (new_name == NULL) {
                TALLOC_FREE(old_name);
-               return WERR_NOMEM;
+               return WERR_NOT_ENOUGH_MEMORY;
        }
 
        if (version != -1 && (version = file_version_is_newer(conn, old_name, new_name)) > 0) {
 
                status = driver_unix_convert(conn, old_name, &smb_fname_old);
                if (!NT_STATUS_IS_OK(status)) {
-                       ret = WERR_NOMEM;
+                       ret = WERR_NOT_ENOUGH_MEMORY;
                        goto out;
                }
 
                /* Setup a synthetic smb_filename struct */
                smb_fname_new = talloc_zero(mem_ctx, struct smb_filename);
                if (!smb_fname_new) {
-                       ret = WERR_NOMEM;
+                       ret = WERR_NOT_ENOUGH_MEMORY;
                        goto out;
                }
 
@@ -955,7 +1230,8 @@ static WERROR move_driver_file_to_download_area(TALLOC_CTX *mem_ctx,
 }
 
 WERROR move_driver_to_download_area(struct auth_session_info *session_info,
-                                   struct spoolss_AddDriverInfoCtr *r)
+                                   struct spoolss_AddDriverInfoCtr *r,
+                                   const char *driver_directory)
 {
        struct spoolss_AddDriverInfo3 *driver;
        struct spoolss_AddDriverInfo3 converted_driver;
@@ -967,7 +1243,7 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
        int i;
        TALLOC_CTX *ctx = talloc_tos();
        int ver = 0;
-       char *oldcwd;
+       struct smb_filename *oldcwd_fname = NULL;
        char *printdollar = NULL;
        int printdollar_snum;
        WERROR err = WERR_OK;
@@ -980,9 +1256,13 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                convert_level_6_to_level3(&converted_driver, r->info.info6);
                driver = &converted_driver;
                break;
+       case 8:
+               convert_level_8_to_level3(&converted_driver, r->info.info8);
+               driver = &converted_driver;
+               break;
        default:
                DEBUG(0,("move_driver_to_download_area: Unknown info level (%u)\n", (unsigned int)r->level));
-               return WERR_UNKNOWN_LEVEL;
+               return WERR_INVALID_LEVEL;
        }
 
        short_architecture = get_short_archi(driver->architecture);
@@ -992,15 +1272,19 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
 
        printdollar_snum = find_service(ctx, "print$", &printdollar);
        if (!printdollar) {
-               return WERR_NOMEM;
+               return WERR_NOT_ENOUGH_MEMORY;
        }
        if (printdollar_snum == -1) {
-               return WERR_NO_SUCH_SHARE;
+               return WERR_BAD_NET_NAME;
        }
 
-       nt_status = create_conn_struct(talloc_tos(), &conn, printdollar_snum,
-                                      lp_pathname(printdollar_snum),
-                                      session_info, &oldcwd);
+       nt_status = create_conn_struct_cwd(talloc_tos(),
+                                          server_event_context(),
+                                          server_messaging_context(),
+                                          &conn,
+                                          printdollar_snum,
+                                          lp_path(talloc_tos(), printdollar_snum),
+                                          session_info, &oldcwd_fname);
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0,("move_driver_to_download_area: create_conn_struct "
                         "returned %s\n", nt_errstr(nt_status)));
@@ -1026,12 +1310,12 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                short_architecture,
                                driver->version);
        if (!new_dir) {
-               err = WERR_NOMEM;
+               err = WERR_NOT_ENOUGH_MEMORY;
                goto err_exit;
        }
        nt_status = driver_unix_convert(conn, new_dir, &smb_dname);
        if (!NT_STATUS_IS_OK(nt_status)) {
-               err = WERR_NOMEM;
+               err = WERR_NOT_ENOUGH_MEMORY;
                goto err_exit;
        }
 
@@ -1072,7 +1356,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                                        driver->driver_path,
                                                        short_architecture,
                                                        driver->version,
-                                                       ver);
+                                                       ver,
+                                                       driver_directory);
                if (!W_ERROR_IS_OK(err)) {
                        goto err_exit;
                }
@@ -1086,7 +1371,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                                                driver->data_file,
                                                                short_architecture,
                                                                driver->version,
-                                                               ver);
+                                                               ver,
+                                                               driver_directory);
                        if (!W_ERROR_IS_OK(err)) {
                                goto err_exit;
                        }
@@ -1102,7 +1388,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                                                driver->config_file,
                                                                short_architecture,
                                                                driver->version,
-                                                               ver);
+                                                               ver,
+                                                               driver_directory);
                        if (!W_ERROR_IS_OK(err)) {
                                goto err_exit;
                        }
@@ -1119,7 +1406,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                                                driver->help_file,
                                                                short_architecture,
                                                                driver->version,
-                                                               ver);
+                                                               ver,
+                                                               driver_directory);
                        if (!W_ERROR_IS_OK(err)) {
                                goto err_exit;
                        }
@@ -1144,7 +1432,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
                                                                        driver->dependent_files->string[i],
                                                                        short_architecture,
                                                                        driver->version,
-                                                                       ver);
+                                                                       ver,
+                                                                       driver_directory);
                                if (!W_ERROR_IS_OK(err)) {
                                        goto err_exit;
                                }
@@ -1160,7 +1449,8 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
        TALLOC_FREE(smb_dname);
 
        if (conn != NULL) {
-               vfs_ChDir(conn, oldcwd);
+               vfs_ChDir(conn, oldcwd_fname);
+               TALLOC_FREE(oldcwd_fname);
                SMB_VFS_DISCONNECT(conn);
                conn_free(conn);
        }
@@ -1174,16 +1464,14 @@ WERROR move_driver_to_download_area(struct auth_session_info *session_info,
 ****************************************************************************/
 
 bool printer_driver_in_use(TALLOC_CTX *mem_ctx,
-                          const struct auth_session_info *session_info,
-                          struct messaging_context *msg_ctx,
-                           const struct spoolss_DriverInfo8 *r)
+                          struct dcerpc_binding_handle *b,
+                          const struct spoolss_DriverInfo8 *r)
 {
        int snum;
        int n_services = lp_numservices();
-       bool in_use = False;
+       bool in_use = false;
        struct spoolss_PrinterInfo2 *pinfo2 = NULL;
        WERROR result;
-       struct dcerpc_binding_handle *b = NULL;
 
        if (!r) {
                return false;
@@ -1194,29 +1482,19 @@ bool printer_driver_in_use(TALLOC_CTX *mem_ctx,
        /* loop through the printers.tdb and check for the drivername */
 
        for (snum=0; snum<n_services && !in_use; snum++) {
-               if (!lp_snum_ok(snum) || !lp_print_ok(snum)) {
+               if (!lp_snum_ok(snum) || !lp_printable(snum)) {
                        continue;
                }
 
-               if (b == NULL) {
-                       result = winreg_printer_binding_handle(mem_ctx,
-                                                              session_info,
-                                                              msg_ctx,
-                                                              &b);
-                       if (!W_ERROR_IS_OK(result)) {
-                               return false;
-                       }
-               }
-
                result = winreg_get_printer(mem_ctx, b,
-                                           lp_servicename(snum),
+                                           lp_servicename(talloc_tos(), snum),
                                            &pinfo2);
                if (!W_ERROR_IS_OK(result)) {
                        continue; /* skip */
                }
 
                if (strequal(r->driver_name, pinfo2->drivername)) {
-                       in_use = True;
+                       in_use = true;
                }
 
                TALLOC_FREE(pinfo2);
@@ -1225,7 +1503,7 @@ bool printer_driver_in_use(TALLOC_CTX *mem_ctx,
        DEBUG(10,("printer_driver_in_use: Completed search through ntprinters.tdb...\n"));
 
        if ( in_use ) {
-               struct spoolss_DriverInfo8 *driver;
+               struct spoolss_DriverInfo8 *driver = NULL;
                WERROR werr;
 
                DEBUG(5,("printer_driver_in_use: driver \"%s\" is currently in use\n", r->driver_name));
@@ -1233,26 +1511,31 @@ bool printer_driver_in_use(TALLOC_CTX *mem_ctx,
                /* we can still remove the driver if there is one of
                   "Windows NT x86" version 2 or 3 left */
 
-               if (!strequal("Windows NT x86", r->architecture)) {
+               if (strequal(SPOOLSS_ARCHITECTURE_NT_X86, r->architecture)) {
+                       if (r->version == 2) {
+                               werr = winreg_get_driver(mem_ctx, b,
+                                                        r->architecture,
+                                                        r->driver_name,
+                                                        3, &driver);
+                       } else if (r->version == 3) {
+                               werr = winreg_get_driver(mem_ctx, b,
+                                                        r->architecture,
+                                                        r->driver_name,
+                                                        2, &driver);
+                       } else {
+                               DBG_ERR("Unknown driver version (%d)\n",
+                                       r->version);
+                               werr = WERR_UNKNOWN_PRINTER_DRIVER;
+                       }
+               } else if (strequal(SPOOLSS_ARCHITECTURE_x64, r->architecture)) {
                        werr = winreg_get_driver(mem_ctx, b,
-                                                "Windows NT x86",
+                                                SPOOLSS_ARCHITECTURE_NT_X86,
                                                 r->driver_name,
                                                 DRIVER_ANY_VERSION,
                                                 &driver);
-               } else if (r->version == 2) {
-                       werr = winreg_get_driver(mem_ctx, b,
-                                                "Windows NT x86",
-                                                r->driver_name,
-                                                3, &driver);
-               } else if (r->version == 3) {
-                       werr = winreg_get_driver(mem_ctx, b,
-                                                "Windows NT x86",
-                                                r->driver_name,
-                                                2, &driver);
                } else {
-                       DEBUG(0, ("printer_driver_in_use: ERROR!"
-                                 " unknown driver version (%d)\n",
-                                 r->version));
+                       DBG_ERR("Unknown driver architecture: %s\n",
+                               r->architecture);
                        werr = WERR_UNKNOWN_PRINTER_DRIVER;
                }
 
@@ -1260,7 +1543,7 @@ bool printer_driver_in_use(TALLOC_CTX *mem_ctx,
 
                if ( W_ERROR_IS_OK(werr) ) {
                        /* it's ok to remove the driver, we have other architctures left */
-                       in_use = False;
+                       in_use = false;
                        talloc_free(driver);
                }
        }
@@ -1413,18 +1696,16 @@ static bool trim_overlap_drv_files(TALLOC_CTX *mem_ctx,
 ****************************************************************************/
 
 bool printer_driver_files_in_use(TALLOC_CTX *mem_ctx,
-                                const struct auth_session_info *session_info,
-                                struct messaging_context *msg_ctx,
+                                struct dcerpc_binding_handle *b,
                                 struct spoolss_DriverInfo8 *info)
 {
        int                             i;
-       uint32                          version;
+       uint32_t                                version;
        struct spoolss_DriverInfo8      *driver;
        bool in_use = false;
        uint32_t num_drivers;
        const char **drivers;
        WERROR result;
-       struct dcerpc_binding_handle *b;
 
        if ( !info )
                return False;
@@ -1437,14 +1718,6 @@ bool printer_driver_files_in_use(TALLOC_CTX *mem_ctx,
 
        /* get the list of drivers */
 
-       result = winreg_printer_binding_handle(mem_ctx,
-                                              session_info,
-                                              msg_ctx,
-                                              &b);
-       if (!W_ERROR_IS_OK(result)) {
-               return false;
-       }
-
        result = winreg_get_driver_list(mem_ctx, b,
                                        info->architecture, version,
                                        &num_drivers, &drivers);
@@ -1493,20 +1766,29 @@ bool printer_driver_files_in_use(TALLOC_CTX *mem_ctx,
 }
 
 static NTSTATUS driver_unlink_internals(connection_struct *conn,
-                                       const char *name)
+                                       const char *short_arch,
+                                       int vers,
+                                       const char *fname)
 {
+       TALLOC_CTX *tmp_ctx = talloc_new(conn);
        struct smb_filename *smb_fname = NULL;
-       NTSTATUS status;
+       char *print_dlr_path;
+       NTSTATUS status = NT_STATUS_NO_MEMORY;
 
-       status = create_synthetic_smb_fname(talloc_tos(), name, NULL, NULL,
-           &smb_fname);
-       if (!NT_STATUS_IS_OK(status)) {
-               return status;
+       print_dlr_path = talloc_asprintf(tmp_ctx, "%s/%d/%s",
+                                        short_arch, vers, fname);
+       if (print_dlr_path == NULL) {
+               goto err_out;
        }
 
-       status = unlink_internals(conn, NULL, 0, smb_fname, false);
+       smb_fname = synthetic_smb_fname(tmp_ctx, print_dlr_path, NULL, NULL, 0);
+       if (smb_fname == NULL) {
+               goto err_out;
+       }
 
-       TALLOC_FREE(smb_fname);
+       status = unlink_internals(conn, NULL, 0, smb_fname, false);
+err_out:
+       talloc_free(tmp_ctx);
        return status;
 }
 
@@ -1519,12 +1801,10 @@ static NTSTATUS driver_unlink_internals(connection_struct *conn,
 bool delete_driver_files(const struct auth_session_info *session_info,
                         const struct spoolss_DriverInfo8 *r)
 {
-       int i = 0;
-       char *s;
-       const char *file;
+       const char *short_arch;
        connection_struct *conn;
        NTSTATUS nt_status;
-       char *oldcwd;
+       struct smb_filename *oldcwd_fname = NULL;
        char *printdollar = NULL;
        int printdollar_snum;
        bool ret = false;
@@ -1544,9 +1824,13 @@ bool delete_driver_files(const struct auth_session_info *session_info,
                return false;
        }
 
-       nt_status = create_conn_struct(talloc_tos(), &conn, printdollar_snum,
-                                      lp_pathname(printdollar_snum),
-                                      session_info, &oldcwd);
+       nt_status = create_conn_struct_cwd(talloc_tos(),
+                                          server_event_context(),
+                                          server_messaging_context(),
+                                          &conn,
+                                          printdollar_snum,
+                                          lp_path(talloc_tos(), printdollar_snum),
+                                          session_info, &oldcwd_fname);
        if (!NT_STATUS_IS_OK(nt_status)) {
                DEBUG(0,("delete_driver_files: create_conn_struct "
                         "returned %s\n", nt_errstr(nt_status)));
@@ -1572,55 +1856,40 @@ bool delete_driver_files(const struct auth_session_info *session_info,
                goto err_out;
        }
 
-       /* now delete the files; must strip the '\print$' string from
-          fron of path                                                */
+       short_arch = get_short_archi(r->architecture);
+       if (short_arch == NULL) {
+               DEBUG(0, ("bad architecture %s\n", r->architecture));
+               ret = false;
+               goto err_out;
+       }
+
+       /* now delete the files */
 
        if (r->driver_path && r->driver_path[0]) {
-               if ((s = strchr(&r->driver_path[1], '\\')) != NULL) {
-                       file = s;
-                       DEBUG(10,("deleting driverfile [%s]\n", s));
-                       driver_unlink_internals(conn, file);
-               }
+               DEBUG(10,("deleting driverfile [%s]\n", r->driver_path));
+               driver_unlink_internals(conn, short_arch, r->version, r->driver_path);
        }
 
        if (r->config_file && r->config_file[0]) {
-               if ((s = strchr(&r->config_file[1], '\\')) != NULL) {
-                       file = s;
-                       DEBUG(10,("deleting configfile [%s]\n", s));
-                       driver_unlink_internals(conn, file);
-               }
+               DEBUG(10,("deleting configfile [%s]\n", r->config_file));
+               driver_unlink_internals(conn, short_arch, r->version, r->config_file);
        }
 
        if (r->data_file && r->data_file[0]) {
-               if ((s = strchr(&r->data_file[1], '\\')) != NULL) {
-                       file = s;
-                       DEBUG(10,("deleting datafile [%s]\n", s));
-                       driver_unlink_internals(conn, file);
-               }
+               DEBUG(10,("deleting datafile [%s]\n", r->data_file));
+               driver_unlink_internals(conn, short_arch, r->version, r->data_file);
        }
 
        if (r->help_file && r->help_file[0]) {
-               if ((s = strchr(&r->help_file[1], '\\')) != NULL) {
-                       file = s;
-                       DEBUG(10,("deleting helpfile [%s]\n", s));
-                       driver_unlink_internals(conn, file);
-               }
+               DEBUG(10,("deleting helpfile [%s]\n", r->help_file));
+               driver_unlink_internals(conn, short_arch, r->version, r->help_file);
        }
 
-       /* check if we are done removing files */
-
        if (r->dependent_files) {
+               int i = 0;
                while (r->dependent_files[i] && r->dependent_files[i][0]) {
-                       char *p;
-
-                       /* bypass the "\print$" portion of the path */
-
-                       if ((p = strchr(r->dependent_files[i]+1, '\\')) != NULL) {
-                               file = p;
-                               DEBUG(10,("deleting dependent file [%s]\n", file));
-                               driver_unlink_internals(conn, file);
-                       }
-
+                       DEBUG(10,("deleting dependent file [%s]\n", r->dependent_files[i]));
+                       driver_unlink_internals(conn, short_arch, r->version, r->dependent_files[i]);
                        i++;
                }
        }
@@ -1630,7 +1899,8 @@ bool delete_driver_files(const struct auth_session_info *session_info,
        unbecome_user();
  err_free_conn:
        if (conn != NULL) {
-               vfs_ChDir(conn, oldcwd);
+               vfs_ChDir(conn, oldcwd_fname);
+               TALLOC_FREE(oldcwd_fname);
                SMB_VFS_DISCONNECT(conn);
                conn_free(conn);
        }
@@ -1643,7 +1913,7 @@ bool delete_driver_files(const struct auth_session_info *session_info,
        2: file doesn't exist
        3: can't allocate memory
        4: can't free memory
-       5: non existant struct
+       5: non existent struct
 */
 
 /*
@@ -1721,12 +1991,12 @@ void map_job_permissions(struct security_descriptor *sd)
     3)  "printer admins" (may result in numerous calls to winbind)
 
  ****************************************************************************/
-bool print_access_check(const struct auth_session_info *session_info,
-                       struct messaging_context *msg_ctx, int snum,
-                       int access_type)
+WERROR print_access_check(const struct auth_session_info *session_info,
+                         struct messaging_context *msg_ctx, int snum,
+                         int access_type)
 {
        struct spoolss_security_descriptor *secdesc = NULL;
-       uint32 access_granted;
+       uint32_t access_granted;
        size_t sd_size;
        NTSTATUS status;
        WERROR result;
@@ -1737,25 +2007,24 @@ bool print_access_check(const struct auth_session_info *session_info,
 
        /* Always allow root or SE_PRINT_OPERATROR to do anything */
 
-       if (session_info->unix_token->uid == sec_initial_uid()
-           || security_token_has_privilege(session_info->security_token, SEC_PRIV_PRINT_OPERATOR)) {
-               return True;
+       if ((session_info->unix_token->uid == sec_initial_uid())
+           || security_token_has_privilege(session_info->security_token,
+                                           SEC_PRIV_PRINT_OPERATOR)) {
+               return WERR_OK;
        }
 
        /* Get printer name */
 
-       pname = lp_printername(snum);
+       pname = lp_printername(talloc_tos(), snum);
 
        if (!pname || !*pname) {
-               errno = EACCES;
-               return False;
+               return WERR_ACCESS_DENIED;
        }
 
        /* Get printer security descriptor */
 
        if(!(mem_ctx = talloc_init("print_access_check"))) {
-               errno = ENOMEM;
-               return False;
+               return WERR_NOT_ENOUGH_MEMORY;
        }
 
        result = winreg_get_printer_secdesc_internal(mem_ctx,
@@ -1765,8 +2034,7 @@ bool print_access_check(const struct auth_session_info *session_info,
                                            &secdesc);
        if (!W_ERROR_IS_OK(result)) {
                talloc_destroy(mem_ctx);
-               errno = ENOMEM;
-               return False;
+               return WERR_NOT_ENOUGH_MEMORY;
        }
 
        if (access_type == JOB_ACCESS_ADMINISTER) {
@@ -1784,8 +2052,7 @@ bool print_access_check(const struct auth_session_info *session_info,
                                                 false);
                if (!NT_STATUS_IS_OK(status)) {
                        talloc_destroy(mem_ctx);
-                       errno = map_errno_from_nt_status(status);
-                       return False;
+                       return ntstatus_to_werror(status);
                }
 
                map_job_permissions(secdesc);
@@ -1799,24 +2066,9 @@ bool print_access_check(const struct auth_session_info *session_info,
 
        DEBUG(4, ("access check was %s\n", NT_STATUS_IS_OK(status) ? "SUCCESS" : "FAILURE"));
 
-        /* see if we need to try the printer admin list */
-
-        if (!NT_STATUS_IS_OK(status) &&
-           (token_contains_name_in_list(uidtoname(session_info->unix_token->uid),
-                                        session_info->info->domain_name,
-                                        NULL, session_info->security_token,
-                                        lp_printer_admin(snum)))) {
-               talloc_destroy(mem_ctx);
-               return True;
-        }
-
        talloc_destroy(mem_ctx);
 
-       if (!NT_STATUS_IS_OK(status)) {
-               errno = EACCES;
-       }
-
-       return NT_STATUS_IS_OK(status);
+       return ntstatus_to_werror(status);
 }
 
 /****************************************************************************
@@ -1832,7 +2084,7 @@ bool print_time_access_check(const struct auth_session_info *session_info,
        bool ok = False;
        time_t now = time(NULL);
        struct tm *t;
-       uint32 mins;
+       uint32_t mins;
 
        result = winreg_get_printer_internal(NULL, session_info, msg_ctx,
                                    servicename, &pinfo2);
@@ -1845,7 +2097,7 @@ bool print_time_access_check(const struct auth_session_info *session_info,
        }
 
        t = gmtime(&now);
-       mins = (uint32)t->tm_hour*60 + (uint32)t->tm_min;
+       mins = (uint32_t)t->tm_hour*60 + (uint32_t)t->tm_min;
 
        if (mins >= pinfo2->starttime && mins <= pinfo2->untiltime) {
                ok = True;
@@ -1870,7 +2122,22 @@ void nt_printer_remove(TALLOC_CTX *mem_ctx,
        result = winreg_delete_printer_key_internal(mem_ctx, session_info, msg_ctx,
                                           printer, "");
        if (!W_ERROR_IS_OK(result)) {
-               DEBUG(0, ("nt_printer_remove: failed to remove rpinter %s",
-                         printer));
+               DEBUG(0, ("nt_printer_remove: failed to remove printer %s: "
+               "%s\n", printer, win_errstr(result)));
+       }
+}
+
+void nt_printer_add(TALLOC_CTX *mem_ctx,
+                   const struct auth_session_info *session_info,
+                   struct messaging_context *msg_ctx,
+                   const char *printer)
+{
+       WERROR result;
+
+       result = winreg_create_printer_internal(mem_ctx, session_info, msg_ctx,
+                                               printer);
+       if (!W_ERROR_IS_OK(result)) {
+               DEBUG(0, ("nt_printer_add: failed to add printer %s: %s\n",
+                         printer, win_errstr(result)));
        }
 }