s3-printing: remove tdb migration invalid printer name checks

WERR_INVALID_PRINTER_NAME only needed to be handled when printing tdb
migration used spoolss, with winreg such errors are no longer possible.

Signed-off-by: Günther Deschner <gd@samba.org>
Autobuild-User: Günther Deschner <gd@samba.org>
Autobuild-Date: Thu Jul  7 19:15:34 CEST 2011 on sn-devel-104
(cherry picked from commit e5955903fdc5ed6f8660b72db0716d4da25c711c)

diff --git a/source3/printing/nt_printing_migrate_internal.c b/source3/printing/nt_printing_migrate_internal.c
index 84b0c314e0d83e3d216859a01562599c339aba21..5d8edc75f85d05ceeb1a82e11de5a0e0487ea6ee 100644 (file)
--- a/source3/printing/nt_printing_migrate_internal.c
+++ b/source3/printing/nt_printing_migrate_internal.c
@@ -132,12 +132,7 @@ static NTSTATUS migrate_internal(TALLOC_CTX *mem_ctx,
                                                 dbuf.dptr,
                                                 dbuf.dsize);
                        SAFE_FREE(dbuf.dptr);
-                       /* currently no WERR_INVALID_PRINTER_NAME equivalent */
-                       if (NT_STATUS_EQUAL(status,
-                              werror_to_ntstatus(WERR_INVALID_PRINTER_NAME))) {
-                               DEBUG(2, ("Skipping migration for non-existent "
-                                               "printer: %s\n", printer_name));
-                       } else if (!NT_STATUS_IS_OK(status)) {
+                       if (!NT_STATUS_IS_OK(status)) {
                                tdb_close(tdb);
                                return status;
                        }
@@ -164,9 +159,7 @@ static NTSTATUS migrate_internal(TALLOC_CTX *mem_ctx,
                                                 dbuf.dptr,
                                                 dbuf.dsize);
                        SAFE_FREE(dbuf.dptr);
-                       /* currently no WERR_INVALID_PRINTER_NAME equivalent */
-                       if (NT_STATUS_EQUAL(status, werror_to_ntstatus(WERR_INVALID_PRINTER_NAME)) ||
-                           NT_STATUS_EQUAL(status, werror_to_ntstatus(WERR_BADFILE))) {
+                       if (NT_STATUS_EQUAL(status, werror_to_ntstatus(WERR_BADFILE))) {
                                DEBUG(2, ("Skipping secdesc migration for non-existent "
                                                "printer: %s\n", secdesc_name));
                        } else if (!NT_STATUS_IS_OK(status)) {