The branch, master has been updated
       via  4108728... s3-spoolss: Use the spoolss_map_to_os2_driver to modify 
drivername.
       via  7d363b8... s3-spoolss: Create a spoolss_map_to_os2_driver function.
       via  a3b81a9... s3-spoolss: Use better names for set_last_from_to.
      from  9bbf761... s3-net: Added a rather trivial "net printing dump" 
command.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 4108728b7e9e4d53624c81984c4ee8e0e6bc81d9
Author: Andreas Schneider <[email protected]>
Date:   Wed Apr 28 17:04:10 2010 +0200

    s3-spoolss: Use the spoolss_map_to_os2_driver to modify drivername.
    
    Signed-off-by: Günther Deschner <[email protected]>

commit 7d363b8922e7ec740858339aaa0052a990b264fa
Author: Andreas Schneider <[email protected]>
Date:   Wed Apr 28 17:03:41 2010 +0200

    s3-spoolss: Create a spoolss_map_to_os2_driver function.
    
    Signed-off-by: Günther Deschner <[email protected]>

commit a3b81a908c9c866066316b01efe5bb0bc8937a86
Author: Andreas Schneider <[email protected]>
Date:   Mon May 10 15:05:46 2010 +0200

    s3-spoolss: Use better names for set_last_from_to.
    
    Use set_driver_mapping() and get_win_driver(), get_os2_driver().
    
    Signed-off-by: Günther Deschner <[email protected]>

-----------------------------------------------------------------------

Summary of changes:
 source3/include/proto.h               |    1 +
 source3/printing/nt_printing.c        |  147 +++++++++++++++++++++++++++-----
 source3/rpc_server/srv_spoolss_util.c |    5 +
 3 files changed, 130 insertions(+), 23 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/include/proto.h b/source3/include/proto.h
index 1853fa9..c197608 100644
--- a/source3/include/proto.h
+++ b/source3/include/proto.h
@@ -4746,6 +4746,7 @@ WERROR spoolss_create_default_devmode(TALLOC_CTX *mem_ctx,
                                      struct spoolss_DeviceMode **devmode);
 WERROR spoolss_create_default_secdesc(TALLOC_CTX *mem_ctx,
                                      struct spoolss_security_descriptor 
**secdesc);
+WERROR spoolss_map_to_os2_driver(TALLOC_CTX *mem_ctx, const char 
**pdrivername);
 int add_new_printer_key( NT_PRINTER_DATA *data, const char *name );
 int delete_printer_key( NT_PRINTER_DATA *data, const char *name );
 int lookup_printerkey( NT_PRINTER_DATA *data, const char *name );
diff --git a/source3/printing/nt_printing.c b/source3/printing/nt_printing.c
index 40a727d..5b68d0b 100644
--- a/source3/printing/nt_printing.c
+++ b/source3/printing/nt_printing.c
@@ -3851,39 +3851,36 @@ static int unpack_values(NT_PRINTER_DATA *printer_data, 
const uint8 *buf, int bu
 /****************************************************************************
  ***************************************************************************/
 
-static char *last_from;
-static char *last_to;
+static char *win_driver;
+static char *os2_driver;
 
-static const char *get_last_from(void)
+static const char *get_win_driver(void)
 {
-       if (!last_from) {
+       if (win_driver == NULL) {
                return "";
        }
-       return last_from;
+       return win_driver;
 }
 
-static const char *get_last_to(void)
+static const char *get_os2_driver(void)
 {
-       if (!last_to) {
+       if (os2_driver == NULL) {
                return "";
        }
-       return last_to;
+       return os2_driver;
 }
 
-static bool set_last_from_to(const char *from, const char *to)
+static bool set_driver_mapping(const char *from, const char *to)
 {
-       char *orig_from = last_from;
-       char *orig_to = last_to;
+       SAFE_FREE(win_driver);
+       SAFE_FREE(os2_driver);
 
-       last_from = SMB_STRDUP(from);
-       last_to = SMB_STRDUP(to);
+       win_driver = SMB_STRDUP(from);
+       os2_driver = SMB_STRDUP(to);
 
-       SAFE_FREE(orig_from);
-       SAFE_FREE(orig_to);
-
-       if (!last_from || !last_to) {
-               SAFE_FREE(last_from);
-               SAFE_FREE(last_to);
+       if (win_driver == NULL || os2_driver == NULL) {
+               SAFE_FREE(win_driver);
+               SAFE_FREE(os2_driver);
                return false;
        }
        return true;
@@ -3902,10 +3899,10 @@ static void map_to_os2_driver(fstring drivername)
        if (!*mapfile)
                return;
 
-       if (strequal(drivername,get_last_from())) {
+       if (strequal(drivername, get_win_driver())) {
                DEBUG(3,("Mapped Windows driver %s to OS/2 driver %s\n",
-                       drivername,get_last_to()));
-               fstrcpy(drivername,get_last_to());
+                       drivername, get_os2_driver()));
+               fstrcpy(drivername, get_os2_driver());
                return;
        }
 
@@ -3954,7 +3951,7 @@ static void map_to_os2_driver(fstring drivername)
 
                if (strequal(nt_name,drivername)) {
                        DEBUG(3,("Mapped windows driver %s to os2 
driver%s\n",drivername,os2_name));
-                       set_last_from_to(drivername,os2_name);
+                       set_driver_mapping(drivername,os2_name);
                        fstrcpy(drivername,os2_name);
                        TALLOC_FREE(lines);
                        return;
@@ -3964,6 +3961,110 @@ static void map_to_os2_driver(fstring drivername)
        TALLOC_FREE(lines);
 }
 
+/**
+ * @internal
+ *
+ * @brief Map a Windows driver to a OS/2 driver.
+ *
+ * @param[in]  mem_ctx  The memory context to use.
+ *
+ * @param[in,out] pdrivername The drivername of Windows to remap.
+ *
+ * @return              WERR_OK on success, a corresponding WERROR on failure.
+ */
+WERROR spoolss_map_to_os2_driver(TALLOC_CTX *mem_ctx, const char **pdrivername)
+{
+       const char *mapfile = lp_os2_driver_map();
+       char **lines = NULL;
+       const char *drivername;
+       int numlines = 0;
+       int i;
+
+       if (pdrivername == NULL || *pdrivername == NULL || *pdrivername[0] == 
'\0') {
+               return WERR_INVALID_PARAMETER;
+       }
+
+       drivername = *pdrivername;
+
+       if (mapfile[0] == '\0') {
+               return WERR_BADFILE;
+       }
+
+       if (strequal(drivername, get_win_driver())) {
+               DEBUG(3,("Mapped Windows driver %s to OS/2 driver %s\n",
+                       drivername, get_os2_driver()));
+               drivername = talloc_strdup(mem_ctx, get_os2_driver());
+               if (drivername == NULL) {
+                       return WERR_NOMEM;
+               }
+               *pdrivername = drivername;
+               return WERR_OK;
+       }
+
+       lines = file_lines_load(mapfile, &numlines, 0, NULL);
+       if (numlines == 0 || lines == NULL) {
+               DEBUG(0,("No entries in OS/2 driver map %s\n", mapfile));
+               TALLOC_FREE(lines);
+               return WERR_EMPTY;
+       }
+
+       DEBUG(4,("Scanning OS/2 driver map %s\n",mapfile));
+
+       for( i = 0; i < numlines; i++) {
+               char *nt_name = lines[i];
+               char *os2_name = strchr(nt_name, '=');
+
+               if (os2_name == NULL) {
+                       continue;
+               }
+
+               *os2_name++ = '\0';
+
+               while (isspace(*nt_name)) {
+                       nt_name++;
+               }
+
+               if (*nt_name == '\0' || strchr("#;", *nt_name)) {
+                       continue;
+               }
+
+               {
+                       int l = strlen(nt_name);
+                       while (l && isspace(nt_name[l - 1])) {
+                               nt_name[l - 1] = 0;
+                               l--;
+                       }
+               }
+
+               while (isspace(*os2_name)) {
+                       os2_name++;
+               }
+
+               {
+                       int l = strlen(os2_name);
+                       while (l && isspace(os2_name[l-1])) {
+                               os2_name[l-1] = 0;
+                               l--;
+                       }
+               }
+
+               if (strequal(nt_name, drivername)) {
+                       DEBUG(3,("Mapped Windows driver %s to OS/2 driver 
%s\n",drivername,os2_name));
+                       set_driver_mapping(drivername, os2_name);
+                       drivername = talloc_strdup(mem_ctx, os2_name);
+                       TALLOC_FREE(lines);
+                       if (drivername == NULL) {
+                               return WERR_NOMEM;
+                       }
+                       *pdrivername = drivername;
+                       return WERR_OK;
+               }
+       }
+
+       TALLOC_FREE(lines);
+       return WERR_OK;
+}
+
 /****************************************************************************
  Get a default printer info 2 struct.
 ****************************************************************************/
diff --git a/source3/rpc_server/srv_spoolss_util.c 
b/source3/rpc_server/srv_spoolss_util.c
index f7877b3..b46fc58 100644
--- a/source3/rpc_server/srv_spoolss_util.c
+++ b/source3/rpc_server/srv_spoolss_util.c
@@ -2180,6 +2180,11 @@ WERROR winreg_get_printer(TALLOC_CTX *mem_ctx,
                goto done;
        }
 
+       /* Fix for OS/2 drivers. */
+       if (get_remote_arch() == RA_OS2) {
+               spoolss_map_to_os2_driver(info2, &info2->drivername);
+       }
+
        if (pinfo2) {
                *pinfo2 = talloc_move(mem_ctx, &info2);
        }


-- 
Samba Shared Repository

Reply via email to