Re: [LOW] STR #4284: ipp backend crash in backendSNMPSupplies()

Michael Sweet <[email protected]> Fri, 8 Mar 2013 10:15:31 -0800 (PST)
Newsgroups gmane.comp.printing.cups.bugs
Message-ID <[email protected]>

--PART-BOUNDARY
Content-Type: text/plain

DO NOT REPLY TO THIS MESSAGE.  INSTEAD, POST ANY RESPONSES TO THE LINK BELOW.

[STR Closed w/Resolution]

Fixed in Subversion repository.

The change is to use the first resolved address, just as for the initial
SNMP query.

Link: https://www.cups.org/str.php?L4284
Version: 1.5.4
Fix Version: 1.7-current (r10887)
--PART-BOUNDARY
Content-Type: text/plain
Content-Disposition: attachment; filename="str4284.patch"

Index: backend/ipp.c
===================================================================
--- backend/ipp.c	(revision 10886)
+++ backend/ipp.c	(working copy)
@@ -2042,7 +2042,8 @@
   */
 
   if (have_supplies &&
-      !backendSNMPSupplies(snmp_fd, http->hostaddr, &page_count, NULL) &&
+      !backendSNMPSupplies(snmp_fd, &(http->addrlist->addr), &page_count,
+                           NULL) &&
       page_count > start_count)
     fprintf(stderr, "PAGE: total %d\n", page_count - start_count);
 

--PART-BOUNDARY
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
cups-bugs mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/cups-bugs

--PART-BOUNDARY--