printing: return last change time with pcap_cache_loaded()

Bug: https://bugzilla.samba.org/show_bug.cgi?id=10652

Signed-off-by: David Disseldorp <ddiss@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>

diff --git a/source3/printing/load.c b/source3/printing/load.c
index 2ba3b2e106d854f4eb5901361aa351655ec4f804..238998d920d7d2b943b01734d4ef78c1e4394cf4 100644 (file)
--- a/source3/printing/load.c
+++ b/source3/printing/load.c
@@ -65,7 +65,7 @@ load automatic printer services from pre-populated pcap cache
 void load_printers(struct tevent_context *ev,
                   struct messaging_context *msg_ctx)
 {
-       SMB_ASSERT(pcap_cache_loaded());
+       SMB_ASSERT(pcap_cache_loaded(NULL));
 
        add_auto_printers();
 

diff --git a/source3/printing/pcap.c b/source3/printing/pcap.c
index 9c44584a4d2edb143d76086f021db4220a6c4403..c5524ad53dbfb3f46364a8d9a2fe154cc45a825e 100644 (file)
--- a/source3/printing/pcap.c
+++ b/source3/printing/pcap.c
@@ -83,13 +83,19 @@ void pcap_cache_destroy_specific(struct pcap_cache **pp_cache)
        *pp_cache = NULL;
 }
 
-bool pcap_cache_loaded(void)
+bool pcap_cache_loaded(time_t *_last_change)
 {
        NTSTATUS status;
        time_t last;
 
        status = printer_list_get_last_refresh(&last);
-       return NT_STATUS_IS_OK(status);
+       if (!NT_STATUS_IS_OK(status)) {
+               return false;
+       }
+       if (_last_change != NULL) {
+               *_last_change = last;
+       }
+       return true;
 }
 
 bool pcap_cache_replace(const struct pcap_cache *pcache)

diff --git a/source3/printing/pcap.h b/source3/printing/pcap.h
index 7dccf84b37497dbf18a2932fabef4f48567ddf59..8fc9e9de31c2aa570ed04fb65d7dbae2584b0a86 100644 (file)
--- a/source3/printing/pcap.h
+++ b/source3/printing/pcap.h
@@ -35,7 +35,7 @@ struct pcap_cache;
 
 bool pcap_cache_add_specific(struct pcap_cache **ppcache, const char *name, const char *comment, const char *location);
 void pcap_cache_destroy_specific(struct pcap_cache **ppcache);
-bool pcap_cache_loaded(void);
+bool pcap_cache_loaded(time_t *_last_change);
 bool pcap_cache_replace(const struct pcap_cache *cache);
 void pcap_printer_fn_specific(const struct pcap_cache *, void (*fn)(const char *, const char *, const char *, void *), void *);
 void pcap_printer_read_fn(void (*fn)(const char *, const char *, const char *, void *), void *);

diff --git a/source3/printing/queue_process.c b/source3/printing/queue_process.c
index aa0d0fb6a894ada96be70a5f52547f85dec3e751..24d361c2bf6741c921547e1404fe579fc5e1bc16 100644 (file)
--- a/source3/printing/queue_process.c
+++ b/source3/printing/queue_process.c
@@ -390,7 +390,7 @@ void printing_subsystem_update(struct tevent_context *ev_ctx,
                               bool force)
 {
        if (background_lpq_updater_pid != -1) {
-               if (pcap_cache_loaded()) {
+               if (pcap_cache_loaded(NULL)) {
                        load_printers(ev_ctx, msg_ctx);
                }
                if (force) {

diff --git a/source3/printing/spoolssd.c b/source3/printing/spoolssd.c
index afb8f4fd2feb7a753984776b8322cd1b7a32519c..b850578e79f27f714250350524da3f06d2da5d0a 100644 (file)
--- a/source3/printing/spoolssd.c
+++ b/source3/printing/spoolssd.c
@@ -304,7 +304,7 @@ static bool spoolss_child_init(struct tevent_context *ev_ctx,
         * If so then we probably missed a message and should load_printers()
         * ourselves. If pcap has not been loaded yet, then ignore, we will get
         * a message as soon as the bq process completes the reload. */
-       if (pcap_cache_loaded()) {
+       if (pcap_cache_loaded(NULL)) {
                load_printers(ev_ctx, msg_ctx);
        }
 
@@ -724,7 +724,7 @@ pid_t start_spoolssd(struct tevent_context *ev_ctx,
         * If pcap has not been loaded yet, then ignore, as we will reload on
         * client enumeration anyway.
         */
-       if (pcap_cache_loaded()) {
+       if (pcap_cache_loaded(NULL)) {
                load_printers(ev_ctx, msg_ctx);
        }