The `cupsGetDests` function incorrectly returned an empty list of printers if there was

no default printer (Issue #5046)

diff --git a/CHANGES.md b/CHANGES.md
index 5d2bedd5dfd17c847d3860b781429520e7ab78ff..bb895d49faac716d188db7e309ef462d8cf776cd 100644 (file)
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -1,4 +1,4 @@
-CHANGES - 2.2.5 - 2017-07-04
+CHANGES - 2.2.5 - 2017-07-07
 ============================
 
 CHANGES IN CUPS V2.2.5
@@ -6,6 +6,8 @@ CHANGES IN CUPS V2.2.5
 
 - Fixed a typo in the CUPS Programming Manual (Issue #5042)
 - Fixed RPM packaging issue (Issue #5043, Issue #5044)
+- The `cupsGetDests` function incorrectly returned an empty list of printers if
+  there was no default printer (Issue #5046)
 - Fixed an issue with Chinese localizations on macOS (rdar://32419311)
 
 

diff --git a/cups/dest.c b/cups/dest.c
index 9d7426ed359d6d975954faa69a61d436997d0954..09bb76b73f1f2e1a099ac330560e0bb455692cfb 100644 (file)
--- a/cups/dest.c
+++ b/cups/dest.c
@@ -2072,17 +2072,6 @@ cupsGetDests2(http_t      *http, /* I - Connection to server or @code CUPS_HTTP_
 
   cupsEnumDests(0, 1000, NULL, 0, 0, (cups_dest_cb_t)cups_get_cb, &data);
 
-  if (cupsLastError() >= IPP_STATUS_REDIRECTION_OTHER_SITE)
-  {
-    DEBUG_printf(("1cupsGetDests2: cupsLastError() is %s, returning 0.", cupsLastErrorString()));
-
-    cupsFreeDests(data.num_dests, data.dests);
-
-    *dests = (cups_dest_t *)0;
-
-    return (0);
-  }
-
  /*
   * Make a copy of the "real" queues for a later sanity check...
   */