Releases: OpenPrinting/cups-browsed
cups-browsed 2.1.0
Security bug fix release. Early release of 2.1.0 to fix CVE-2024-47176 and CVE-2024-47850.
New features:
- Removed support for legacy CUPS browsing and for LDAP
Legacy CUPS browsing is not needed any more and, our implementation accepting any UDP packet on port 631, causes vulnerabilities, and our LDAP support is does not comply with RFC 7612 and is therefore limited. Fixes CVE-2024-47176 and CVE-2024-47850
Bug fixes:
- Default
BrowseRemoteProtocols
should not includecups
protocol
Works around CVE-2024-47176 and CVE-2024-47850, the fix is the complete removal of legacy CUPS Browsing functionality - Do not generate PPD for remote raw queues
If the destination queue is raw, the local queue generated by cups-browsed should also be raw (Pull request #44). daemon/cups-browsed.service
: Addsystem-cups.slice
Thesystem-cups.slice
file is not required. Thesystem-cups
slice will be automatically created if the file is missing (cups Pull request #1035, Pull request #35).cups-browsed.c
: Remove duplicate#include ...
Pull request #40
cups-browsed 2.0.1
Bug fix release: Collected fixes of one year, mainly reliability issues.
- Ignore attributes with
IPP_TAG_NOVALUE
We don't need to record attributes withIPP_TAG_NOVALUE
value tag, cupsd will decide what to use based on the incoming document (Pull request #33). - Ignore attributes with empty values
Previously, if IPP attribute value was empty, cups-browsed saved the attribute as printer option "=0", e.g.orientation-requested=0
. Often0
is invalid and as no cluster member printer has this value therefore a job with this value gets rejected (Pull request #32). - README.md: Use MarkDown for link to bug tracker
(Pull request #31) - Initialize variables to not get used uninitialized
(Pull request #26) - Fix memory leak found by valgrind
(Pull request #26) - Fix the daemon crash when
get-printer-attributes
fails
cups-browsed crashed when it found a remote CUPS queue shared by mDNS on local network, but IPP request for this queue failed. The emptyprattrs
data structure is later accessed, which causes the crash (Pull request #25). - Better handle damage of queues created by cups-browsed
cups-browsed-generated print queues are marked with acups-browsed=true
attribute. Sometimes it gets lost and the queue not considered cups-browsed-generated any more. As queues withimplicitclass://...
device URI are always from cups-browsed, ignore the missing flag and always restore them as cups-browsed queues, so that cups-browsed assigns a destination printer for each job (Issue #429) - Fix build with Avahi disabled
(Issue #19, Pull request #20)
cups-browsed 2.0.0
Final 2.0.0 release with some last fixes
- Ensure we always send a valid name to
remove_bad_chars()
In case the found queue is a CUPS remote queue shared via DNS-SD, therp_value
can be without '/', which leads tocups-browsed
crash if it is set to create the local queue based on the remote name (Pull request #13, Fedora Bugzilla #2150035). - Use description/location from server if available, otherwise from client
When we create a local queue we first check whether we actually got description and location strings from the remote server/printer, if they are empty we do not set empty strings but use the IPP attributes saved locally for our local queue. This way, if the server does not provide description/location and the user sets their own, that one is conserved through reboots and daemon restarts (Issue #16).
cups-browsed 2.0rc2
Some final, important bug fixes before the release: 100% CPU after failed printer access, broken print-quality-default
attribute.
- Fixed cups-browsed getting stuck in busy loop
When the functioncreate_queue()
fails to create a local print queue and the failure is not intermittent, it sets a global variable to stop the main thread's loop for updating local queues. With the variable not reset no queue updates happened ever again and cups-browsed fell into a busy loop taking up to 100% CPU. We have solved this by doing away with the variable and simply mark these printers as disappeared (Ubuntu bug #2018504). - Do not record
*-default
IPP attributes of local CUPS queues
Many of the*-default
IPP attributes represent properties already covered by the PPD option defaults which we also record. In addition, there is alsoprint-quality-default
where IPP reportsdraft
,normal
, andhigh
settings while CUPS only accepts3
,4
, and5
, and on everything else it setsprint-quality-default=0
which is invalid and jobs do not get printed. So we stop saving and loading these attributes. - Build system: Removed unnecessary lines in
Makefile.am
Removed theTESTdir
andTEST_SCRIPTS
entries inMakefile.am
. They are not needed and letmake install
try to installrun-tests.sh
in the source directory, where it already is, causing an error. run-tests.sh
: Use pkgconfig instead of deprecated cups-config (Pull request #9).
cups-browsed 2.0rc1
Release Candidate: Prefer Apple Raster against PDF, and fix implicitclass
backend crash.
- Prefer sending jobs in Apple Raster instead of in PDF
If a destination printer supports both PDF and Apple Raster, and if it is not a remote CUPS queue, prefer sending the job in Apple Raster, as printers print this more reliably (Ubuntu bug 2014976) run-tests.sh
: Let emulated printers support PDF input
To test that cups-browsed prefers Apple Raster when the printer supports both PDF and Apple Raster as input format, we let the printers emulated byippeveprinter
also support PDF.implicitclass
backend: NULL-initialize filter data field for Raster header
We are running a filter chain without PPD file, so we do not have Raster header, so initialize it to NULL.
cups-browsed 2.0b4
In this forth beta we have added a test script, to give make check
functionality but also for using it in the autopkgtests of debian/Ubuntu and for general CI. Found and fixed many bugs in cups-browsed itself during the development of the script.
- Added test script for
make test
/make check
, CI, autopkgtest, ...
The script test/run-tests.sh creates emulations of IPP printers viaippeveprinter
(of CUPS 2.x) and checks whether cups-browsed creates corresponding CUPS queues, whether a job to such a queue gets actually printed, and whether cups-browsed removes the queues again when the printers are shut down. implicitclass
backend: If no destination got reported by cups-browsed, retry after one minute, not the standard 5 minutes of CUPS.debug_printf()
: Check for need of log rotation only if log file is set and opened, to avoid a crash.on_printer_modified()
: Added NULL check to avoid a crash.ipp_discoveries_add()
: Ignore duplicate entries.
These are most probably caused by a bug in Avahi, having certain discoveries of a printer reported twice and others not. When the printer disappears Avahi reports the disappearal of each discovery correctly, leaving the duplicate entry untreated (removing only one instance of it) and cups-browsed assumes that the printer is still there, keeping its CUPS queue.update_cups_queues()
: Reset counter for pausing CUPS queue updates.
Otherwise after having updated the number of queues supposed to be the maximum for one run ofupdate_cups_queues()
, cups-browsed will never update any queue again.resolve_callback()
/resolver_wrapper()
: New thread only when printer found
We move the check which resolver event we have (found/failure) already in the main thread (resolver_wrapper()
) and launch a new thread only if we have found a new printer and have to investigate whether to add a queue for it or not.resolve_callback()
only initiates this investigation now. This way we do not need to pass the resolver data structure (of typeAvahiServiceResolver*
) into the new thread, which caused segfaults.create_remote_printer_entry()
: Corrected some memory freeing when a printer data structure is deleted, but this has not caused a segfault in the recent tests.- Fixed issues reported by Red Hat Coverity tool (Pull request #6)
configure.ac
: Change deprecatedAC_PROG_LIBTOOL
forLT_INIT
(Pull request #5)configure.ac
: cups-browsed doesn't need C++
cups-browsed 2.0b3
Fixes of bugs found while packaging for distros, in the build system and the source code documentation
COPYING
,NOTICE
: Simplification for autotools-generated files
autotools-generated files can be included under the license of the upstream code, and FSF copyright added to upstream copyright list. SimplifiedCOPYING
appropriately.Makefile.am
: IncludeLICENSE
in distribution tarball
cups-browsed 2.0b2
Second beta release approaching cups-browsed 2.0.0: Tons of fixes in source code documentation and build system
- cups-browsed is not part of cups-filters any more. Reflect this in screen messages and comments in
cups-browsed.c
. Makefile.am
: IncludeNOTICE
in distribution tarballconfigure.ac
: Addedforeign
to theAM_INIT_AUTOMAKE()
call. Makes automake not require a file namedREADME
.- Cleaned up
.gitignore
- Tons of fixes in the source code documentation:
README.md
,INSTALL
,DEVELOPING.md
,CONTRIBUTING.md
,COPYING
,NOTICE
, ... Adapted to the cups-browsed component, added links.
cups-browsed 2.0b1
First beta release approaching cups-browsed 2.0.0.
- Added multi-threaded operation, the Avahi resolver callback (which examines the remote printer, registers it, checks whether we want a local queue for it, adds it to a cluster, ...) and the creation/modification of a local CUPS queue is now done in separate threads, so that these processes can get executed in parallel to keep the local queues up-to-date more timely and to not overload the system's resources. Thanks a lot to Mohit Mohan who did this work as Google Summer of Code 2020 project.
- Let the implicitclass backend use filter functions instead of calling filter executables.
- Build system, README.md: Require CUPS 2.2.2+. Removed now unneeded ./configure switches for old CUPS versions.
- Build system: Remove '-D_PPD_DEPRECATED=""' from the compiling command lines of the source files which use libcups. The flag is not supported any more for longer times already and all the PPD-related functions deprecated by CUPS have moved into libppd now.
- Build system: Add files in gitignore that are generated by "autogen.sh", "configure", and "make" (Pull request #336).
- implicitclass: Added "#include <signal.h>" (Issue #335).