Created attachment 13509 [details] For v4-4-test Printers published in Active Directory are deleted by the pruning task running on domain controllers. This task pings the print server each eight hours to check the printer availability and prune it from the directory if the printer is no longer available. Commit a4157e7c5d75 removed the braces from the printer GUID in the GetPrinter info level 7 response, but MS-RPRN section 2.2 says this protocol uses curly braced string representation.
Comment on attachment 13509 [details] For v4-4-test Would have preferred to avoid the double strdup, but still looks okay as-is.
Created attachment 13620 [details] Patch for master The double strdup is necessary to convert the GUID string to uppercase, otherwise the printers are pruned. New patch attached freeing the temporary copy.
Patch looks good to me. Can you submit to samba-technical@lists.samba.org so David and I can +1 and push it ? Thanks, Jeremy.
Created attachment 13661 [details] Patch for v4-4
Created attachment 13662 [details] Patch for v4-5
Created attachment 13663 [details] Patch for v4-6
Created attachment 13664 [details] Patch for v4-7
Reassigning to Karolin for inclusion in 4.7.next, 4.6.next.
(In reply to Jeremy Allison from comment #8) Pushed to autobuild-v4-{7,6}-test.
(In reply to Karolin Seeger from comment #9) Pushed to both branches. Closing out bug report. Thanks!