programming error? in snmp.c

[michaelrsweet]

Version: 1.2.6
CUPS.org User: kmuto.debian

Hi,
This STR is as same as Debian#397833,

backend/snmp.c line 1755-
if (http);
{
/*
* IPP is supported...
*/

It looks like the bracketed block is ment to be conditional on the
value of 'http'. But since there is this strange semicolon at the end
of the line with the 'if', the block is executed unconditionally.
I think that this is a bug and that the semicolon should be removed.

[michaelrsweet]

CUPS.org User: mike

It is an error, but thanks to how the CUPS APIs check for NULL values it just introduces inefficiency (building an IPP request that is never used) rather than crashing...

Fixed in trunk and 1.2...