[PATCH] Renaming a printer fails in Printer Management Console

Jeremy Allison jra at samba.org
Mon Aug 31 22:27:14 UTC 2015


On Tue, Sep 01, 2015 at 12:06:58AM +0200, David Disseldorp wrote:
> Hi Garming,
> 
> On Fri, 28 Aug 2015 14:06:31 +1200, Garming Sam wrote:
> 
> > Hi,
> > 
> > This is patch (with some additional allocation checks) I wrote late last 
> > year that was somewhat forgotten about. It fixes bug 10770, which 
> > explains that Samba doesn't allow renaming of a printer using the F2 key.
> > 
> > What was missing was the info level 4 call of _spoolss_SetPrinter, and 
> > here I've just retrieved the old printer, filled it in and used the 
> > update_printer_call which takes struct info level 2. I previously tried 
> > removing some of the additional talloc calls, but it seemed more trouble 
> > than it was worth at the time with some parts freeing somewhat unexpectedly.
> > 
> > Please review and push.
> > 
> > https://bugzilla.samba.org/show_bug.cgi?id=10770
> 
> Looks like I'm a little late here. Nevertheless, it looks as though
> there are a few error paths that don't clean up tmp_ctx. Also, it's
> probably worth splitting this logic into a separate function.
> Otherwise I think it looks good, and these minor nits can be addressed
> after Jeremy's push goes in.

Damn, I missed those, sorry. Might clean and re-push...

> Thanks for your work on this!
> 
> Cheers, David
> 



More information about the samba-technical mailing list