return NT_STATUS_IS_OK(status);
}
-void pcap_cache_replace(const struct pcap_cache *pcache)
+bool pcap_cache_replace(const struct pcap_cache *pcache)
{
const struct pcap_cache *p;
+ NTSTATUS status;
+
+ status = printer_list_mark_reload();
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0, ("Failed to mark printer list for reload!\n"));
+ return false;
+ }
for (p = pcache; p; p = p->next) {
pcap_cache_add(p->name, p->comment);
}
+
+ status = printer_list_clean_old();
+ if (!NT_STATUS_IS_OK(status)) {
+ DEBUG(0, ("Failed to cleanup printer list!\n"));
+ return false;
+ }
+
+ return true;
}
void pcap_cache_reload(struct tevent_context *ev,
done:
DEBUG(3, ("reload status: %s\n", (pcap_reloaded) ? "ok" : "error"));
- if (pcap_reloaded) {
+ if ((pcap_reloaded) && (post_cache_fill_fn_handled == false)) {
/* cleanup old entries only if the operation was successful,
* otherwise keep around the old entries until we can
* successfuly reaload */
if (!NT_STATUS_IS_OK(status)) {
DEBUG(0, ("Failed to cleanup printer list!\n"));
}
- if ((post_cache_fill_fn_handled == false)
- && (post_cache_fill_fn != NULL)) {
+ if (post_cache_fill_fn != NULL) {
post_cache_fill_fn(ev, msg_ctx);
}
}
void pcap_cache_destroy_specific(struct pcap_cache **ppcache);
bool pcap_cache_add(const char *name, const char *comment);
bool pcap_cache_loaded(void);
-void pcap_cache_replace(const struct pcap_cache *cache);
+bool pcap_cache_replace(const struct pcap_cache *cache);
void pcap_printer_fn_specific(const struct pcap_cache *, void (*fn)(const char *, const char *, void *), void *);
void pcap_printer_fn(void (*fn)(const char *, const char *, void *), void *);
TALLOC_FREE(frame);
if (tmp_pcap_cache) {
+ bool ret;
/* We got a namelist, replace our local cache. */
pcap_cache_destroy_specific(&local_pcap_copy);
local_pcap_copy = tmp_pcap_cache;
/* And the systemwide pcap cache. */
- pcap_cache_replace(local_pcap_copy);
+ ret = pcap_cache_replace(local_pcap_copy);
+ if (!ret)
+ DEBUG(0, ("failed to replace pcap cache\n"));
/* Caller may have requested post cache fill callback */
- if (cb_args->post_cache_fill_fn != NULL) {
+ if (ret && cb_args->post_cache_fill_fn != NULL) {
cb_args->post_cache_fill_fn(cb_args->event_ctx,
cb_args->msg_ctx);
}
return false;
}
} else {
- /* Replace the system cache with our
- * local copy. */
- pcap_cache_replace(local_pcap_copy);
-
DEBUG(10,("cups_cache_reload: async read on fd %d\n",
*p_pipe_fd ));