[cups.bugs] [LOW] STR #4153: <DefaultPrinter> tag in printers.conf closes with </Printer>

Dominic Cleal dominic at computerkb.co.uk
Thu Aug 2 09:33:25 PDT 2012


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

[STR New]

While testing support for parsing printers.conf in Augeas
(http://augeas.net, a configuration API), it was found the
<DefaultPrinter> section is closed with </Printer> rather than the more
obvious </DefaultPrinter>.  The Augeas lens was expecting a matching set
of tags so failed to parse.

While this is an issue with Augeas (though not an easy one to fix), it
seems this could be a common trap and it improved by using
</DefaultPrinter> to match HTML, XML and Apache HTTP type config files.

Attached is a patch to change this, preserving compatibility to read old
printers.conf files.  It's untested, but compiles with trunk.

Link: http://www.cups.org/str.php?L4153
Version: 1.6-current
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: r10562_defaultprinter.patch
URL: <http://lists.cups.org/pipermail/cups-devel/attachments/20120802/dbc9e2e3/attachment.ksh>


More information about the cups-devel mailing list