Attachment 10950 Details for Bug 11210 – patch for 4.1.next

[patch] patch for 4.1.next

bso11210_stale_printer_name_cache_41t.patch (text/plain), 1.83 KB, created by David Disseldorp on 2015-04-14 09:21:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Disseldorp

Created: 2015-04-14 09:21:31 UTC

Size: 1.83 KB

patch

obsolete

>From 01969a0bfa6b4e678d93915667b50f6a443e6bbf Mon Sep 17 00:00:00 2001
>From: David Disseldorp <ddiss@samba.org>
>Date: Wed, 1 Apr 2015 01:03:13 +0200
>Subject: [PATCH] spoolss: purge the printer name cache on name change
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Currently the name cache is only cleared on printer deletion. This means
>that if a printer undergoes a name change, the old name remains in the
>cache and can be subsequently used incorrecly if another printer takes
>the same name as the old.
>
>Bug: https://bugzilla.samba.org/show_bug.cgi?id=11210
>
>Reported-by: Franz PfÃ¶rtsch <franz.pfoertsch@brose.com>
>Signed-off-by: David Disseldorp <ddiss@samba.org>
>Reviewed-by: Jeremy Allison <jra@samba.org>
>
>Autobuild-User(master): Jeremy Allison <jra@samba.org>
>Autobuild-Date(master): Tue Apr 14 05:37:50 CEST 2015 on sn-devel-104
>
>(cherry picked from commit a97507a9a7ba01beead6a621e1210618e93a9f9c)
>---
> source3/rpc_server/spoolss/srv_spoolss_nt.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/source3/rpc_server/spoolss/srv_spoolss_nt.c b/source3/rpc_server/spoolss/srv_spoolss_nt.c
>index 34f0b57..42caff1 100644
>--- a/source3/rpc_server/spoolss/srv_spoolss_nt.c
>+++ b/source3/rpc_server/spoolss/srv_spoolss_nt.c
>@@ -6472,6 +6472,9 @@ static WERROR update_dsspooler(TALLOC_CTX *mem_ctx,
> 						 snum, printer->sharename ?
> 						 printer->sharename : "");
> 		}
>+
>+		/* name change, purge any cache entries for the old */
>+		prune_printername_cache();
> 	}
> 
> 	if (printer->printername != NULL &&
>@@ -6508,6 +6511,9 @@ static WERROR update_dsspooler(TALLOC_CTX *mem_ctx,
> 			notify_printer_printername(server_event_context(),
> 						   msg_ctx, snum, p ? p : "");
> 		}
>+
>+		/* name change, purge any cache entries for the old */
>+		prune_printername_cache();
> 	}
> 
> 	if (printer->portname != NULL &&
>-- 
>2.1.4
>

Flags:

jra: review+

Actions: View

Attachments on bug 11210: 10944 | 10950 | 10951 | 10952