[SCM] Samba Shared Repository - branch master updated

Günther Deschner gd at samba.org
Tue May 18 07:26:45 MDT 2010


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 <asn at samba.org>
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 <gd at samba.org>

commit 7d363b8922e7ec740858339aaa0052a990b264fa
Author: Andreas Schneider <asn at samba.org>
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 <gd at samba.org>

commit a3b81a908c9c866066316b01efe5bb0bc8937a86
Author: Andreas Schneider <asn at samba.org>
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 <gd at samba.org>

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

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


More information about the samba-cvs mailing list