scheduler: Fix regression in background thread (Issue #934)

The con->bg_pending value is used to indicate that the background thread
has finished running and that the connection can be safely closed.

This commit makes the con->bg_pending = 0 only after the connection
can be safely closed.

scheduler: Fix potential bug when adding new printers (Issue #934)

When adding a new printer a thread is created to handle PPD generation,
and the requesting client connection might close while the thread is running.
This commit prevents full connection cleanup until the thread is finished.

Report an error if you use lpadmin to set defaults for a temporary queue (Issue #237)

Fix document-unprintable-error support (Issue #391)

Fix cupsCopyDestInfo device connection detection (Issue #586)

Fix Upgrade header handling when there is no TLS support (Issue #775)

Allow PPD choices and PWG keywords to select media (Issue #238)

scheduler: Fix sending response headers to client

Sometimes headers are not correctly copied into response to the client (some are missing). It happens because sent_header is set prematurely before the actual send happens. The present code in affected cupsdWriteClient() scope looks like code remains from CUPS 1.6.3, where cupsdSendHeader() is called earlier and generates the required headers by itself - the current cupsdSendHeader() sends only the headers which are saved in array fields, so the premature setting of sent_header sometimes causes not having all headers in the response.

With the change, testing via curl gives reliable results all time.

Resolve CI issue on macOS.

Update CUPS-Add-Modify-Printer and CUPS-Create-Local-Printer code to delay responding until the PPD is successfully generated (Issue #347)

Fix backported changes for race condition.

Mirror fix from master/2.5 for potential race condition with CUPS-Create-Local-Printer (Issue #871)

Fix httpGets timeout handling (Issue #879)

Fix the initialization and validation of checkbox and text fields in CGI forms.

cups/ipp.c: Use strcmp, since attribute names are case sensitive

Fix IPv6 address encoding in the Host: header (Issue #903)

ppd-cache.c: Check for required attributes if URF or PWG Raster are found

Some devices have image/urf in document-format-supported, but is missing urf-supported
if AirPrint support is turned off, which breaks PPD generation. Check for attribute
urf-supported when we are about to decide whether the printer uses AirPrint, so in case
the device supports another driverless standard, we can use it for PPD generation.
Apply the same for PWG Raster too.

Fixes Fedora issue #2263053

Use PAM password-auth or system-auth if present

Enables possibility to use PAM modules password-auth or system-auth
if they exist during compilation. password-auth module is
for remote authentication and system-auth for local authentication.

Return HTTP status 404 for non-existing classes and printers (Issue #423)

Merge pull request #888 from alexpevzner/pzz-2.4.x-fix-warnings

Fixed a couple of gcc warnings

Fixed gcc warning

In cups/dest.c:

    dest.c:1830:21: warning: ‘%s’ directive argument is null [-Wformat-overflow=]
     1733 |       DEBUG_printf(("1cupsGetNamedDest: Using name=\"%s\"...", name));
  |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This is really true, and gcc was smart enough to deduct it from context.
Obviously, we meant to print dest_name here.

Fix Oki 407 freeze when printing larger jobs

backend/ipp.c: Fix printing jobs with long names on older IPP printers

On older printers (ones which don't support IPP operation Create-Job)
we concatenate job number and title into one string, which we use as
IPP attribute job-name. If the original title was almost 255 chars,
the joining the strings will overflow maximal required length
for this attribute, and Validate-Job fails.

We could check whether the string is longer than 255 and cut it,
but I chose to shrink the buffer to 256, since we already use snprintf()
which will cut the string and put null terminator for us.

Fixes #644

backend/ipp.c: Fix infinite loop with Kerberos

If IP address was used with Kerberos, IPP backend retried connection
indefinitely. The fix is to abort when we find out hostname is an IP
address and we require Kerberos.

Merge fix for #859 from master

Backport fix for crash in ppdEmitString()

Add/update copyrights

backend/ipp: Fix print-scaling option support for IPP printers

Fixes #862

httpAddrConnect2: Check for error if POLLHUP is in valid revents

Some Linux kernel versions put POLLOUT|POLLHUP into revents when client tries to connect with httpAddrConnect2(), which makes the connection fail.
Let's check the option SO_ERROR before scratching the attempt - if there is no error, remove POLLHUP from revents.

I've re-purposed previously Solaris-only code to be used everywhere if the conditions are met - this should prevent bigger delays than necessary.

Slightly different issue than #827, but with similar symptoms (kernel sending POLLOUT|POLLHUP).

scheduler: Fix build failure on Linux with '-Werror -Wall'

The string which we add into array stays `const` after fixing #814
(because `strdup()`, which caused memory leak returned `char *`)
and compiler fails because of it if it runs with '-Werror -Wall'.

scheduler: Set reasons in cupsdSetJobState only if we call finalize_job later

Before the fix, every successfully printed jobs ended up with
'processing-to-stop-point' as the last reasons message.

Final fix for #828 .

raster-interpreter.c: Fix crash in 'scan_ps()' found by fuzzer

Fuzzer using `_cupsRasterExecPS()` found a way how to pass NULL into
`scan_ps()`, causing crash - we have to sanitize the argument for NULL
to fix it.

Fixes #831

Merge fix for #829

lpstat: Implemenet successful filter for jobs

scheduler/colorman.c: Fix memory leak during creating color profile

Since the array profiles is set to use strdup() as a copy function, we
don't have to use strdup() on the element which is passed as parameter
of cupsArrayAdd() - using the strdup() as we used till now causes
memory leak.

Reproducer is the same as #813

Merge branch '2.4.x' into colorman-leak-2-4

Merge pull request #805 from alexpevzner/2.4.x

cups/ppd-cache.c: Add cupsUrfSupported to generated PPD (see #804)

 scheduler/job.c: unload job before freeing job history in cupsdDeleteJob()

With "PreserveJobHistory Off", LogLevel not set to debug (or debug2),
and "LogDebugHistory 200" (the default), cupsdDeleteJob() frees the
job history and then unloads the job.  However, unload_job() calls
cupsdLogJob() which re-creates the job history and puts "Unloading..."
into it because level (debug) is greater than LogLevel (warn) and
LogDebugHistory is set to 200 messages by default.  Unused (and
unreachable) job history is left behind, resulting in a memory leak.

scheduler/colorman.c: Fix memory leak during creating color profile

 Since the array `profiles` is set to use `strdup()` as a copy function, we
 don't have to use `strdup()` on the element which is passed as parameter
 of `cupsArrayAdd()` - using the `strdup()` as we used till now causes
 memory leak.

 Reproducer is the same as https://github.com/OpenPrinting/cups/pull/813

cups/dest.c: Merge fix for #751 into 2.4

The relevant master commit https://github.com/OpenPrinting/cups/commit/082c5ae12a

cups/dest.c: Merge fix for #751 into 2.4

cups/ppd-cache.c: Add cupsUrfSupported to generated PPD (see #804)

It fixes driverless printing on Pantum BM5100ADW Series and, probably,
on many other devices.

- Without this parameter, /usr/lib/cups/filter/universal generates RGB
image/urf even for monochrome printer
- Pantum BM5100ADW Series rejects to print these RGB images with
the "Print job canceled at printer" status.
- Probably, this issue affects many other devices

See #804 for details.

cups/hash.c: Put support for MacOS/Win SSL libs back

- I mustn't remove their support in patch release - this should happen in 2.5 only.
- I have put back support for several hashes as well - they should be removed in 2.5.
- restrict usage of second block hashing only if OpenSSL/LibreSSL/GnuTLS is available (since IMO I found its implementation useless for SSL libs, which will be removed in 2.5)

This removes hard dependency on OpenSSL/LibreSSL or GnuTLS in 2.4.x, which will happen in 2.5.

cups/hash.c: LibreSSL version does not support several hashes

cups/hash.c: Put support for MacOS/Win SSL libs back

- I mustn't remove their support in patch release - this should happen in
2.5 only.
- I have put back support for several hashes as well - they
should be removed in 2.5.
- restrict usage of second block hashing only if OpenSSL/LibreSSL/GnuTLS
  is available

Prepare files for possible next release

systemv/cancel.c: Merge fix from master

Master commit:
https://github.com/OpenPrinting/cups/commit/c5ad7aa

Prepare release 2.4.7

raster-interpret.c: Fix CVE-2023-4504

We didn't check for end of buffer if it looks there is an escaped
character - check for NULL terminator there and if found, return NULL
as return value and in `ptr`, because a lone backslash is not
a valid PostScript character.

cups/http-addrlist.c: Fix lpstat hanging on Solaris

Fixes #156

cups/http-addrlist.c: Fix lpstat hanging on Solaris

Fixes #156

backen/lpd.c: Fix port reserving behavior for RFC 1179

From master commit:
https://github.com/OpenPrinting/cups/commit/9f0ce570117aa

Fixes #743

Merge fix from master regarding lpstat hanging

Master commit:
https://github.com/OpenPrinting/cups/commit/3f763a960f3

Fixes #773

scheduler/job.c: Merge fix from master for extensive logging

Master commit:
https://github.com/OpenPrinting/cups/commit/541f72d1cc1

Fixes #604

Fix CI failures on MacOS and Windows

- Fix MD5 on MacOS
- Update libressl version to fix Windows failure in CI

Both originate from master commits:
https://github.com/OpenPrinting/cups/commit/26836ad6312665091e5e47f547d2ff5e429ec031
https://github.com/OpenPrinting/cups/commit/60e0f489172c88f6a67a887180d7395c4d8afe52

Try enable CodeQL on 2.4.x

Update libressl version to fix CI failure

Fix MD5 deps on MacOS

Try to enable Github Actions on 2.4.x branch

cups/hash.c: Merge the fix from master

Master commit:
https://github.com/OpenPrinting/cups/commit/4db1ca0ea978f6

Fixes #762

Merge fix from master

Commit in master:
https://github.com/OpenPrinting/cups/commit/b7a60b8c0357

Clean up changelog and normalize reporting of CVEs.

Fix a bug in the copy stack code (Issue #768)

Use ASN1_TIME_to_tm when converting X.509 date/time values.

Update CHANGES.md

Merge lpd fix from master branch

Prepare files for next release in the future

Update CHANGES.md

Fix use-after-free in cupsdAcceptClient() (fixes CVE-2023-34241)

Fix use-after-free when logging warnings in case of failures
in `cupsdAcceptClient()` (fixes CVE-2023-34241)

Merge pull request #735 from AtariDreams/Fixer

Fix compilation on older macOS versions

Fix compilation on older macOS versions

Older versions of macOS do not support SecTrustCopyCertificateChain, so let's revert back to using SecTrustGetCertificateCount.

Fixes https://github.com/OpenPrinting/cups/issues/715

Merge pull request #730 from zdohnal/cupssinglefile_24x

 cups/ppd-cache.c: Put cupsSingleFile into generated PPD

 cups/ppd-cache.c: Put cupsSingleFile into generated PPD

Some printers are not able to print multiple files in one job via IPP
Everywhere. Adding the PPD keyword enables the feature on all printers
in exchange for performance degradation (there is a 1-2s pause between
printed files).

Fixes #643

Prepare files for next release

cups/cups.h: Update for 2.4.5

Finish hotfix release 2.4.5

Merge pull request #727 from AtariDreams/hotfix

Regression: Certificate data is corrupted during base64 conversion

Regression: Certificate data is corrupted during base64 conversion

The bug during which certificates become corrupt was introduced in a521b235a1abc008cb0b2f490a765bb31e2ec14b. It turns out some data was not being truncated after being promoted to an integer, causing 1s to be ORed into the index number when they should not have been.

I only intended to remove the & 255 from the other side, where the & 63 would have rendered that operation redundant.

I apologize for this error. I made the appropriate changes I intended to make in this new PR.

Please merge and make a new release if possible.

Thank you so much!

Prepare files for next release

Prepare for hotfix release 2.4.4

Merge fix for #719 into 2.4.x

Merge configure fix from master, bump version.

Update CHANGES.md for next release

Remove 2.4.4 from CHANGES.md for now

2.4.3

Merge pull request #711 from zdohnal/Plombo-printer-conf-strings

Save strings file path in printers.conf by Bryan Cain + feedback applied

printers.c: Set .strings file path if PPD cache is valid (fixes #710)

Update configure script.

Merge pull request #704 from l1gi/master

host_os_name value in config-scripts has changed to 'solaris'

CI: Fix i386 for now

Although Ubuntu is dropping support for i386, this PR fixes build i386
for now. However we have to remove image upgrade, so probably once there
is a new image, i386 will be broken again.

I would remove the job for i386 once it breaks again due removing
support for i386 in Ubuntu.

host_os_name value in config-scripts has changed to 'solaris'

host_os_name value in config-scripts has changed to 'solaris'

Update CHANGES.md

cups/string.c: Return if `size` is 0 (fixes CVE-2023-32324)

SECURITY.md: Use GitHub private advisories for reporting (fixes #703)

CHANGES.md: Add pull requests which fix noticeable bugs

Merge pull request #705 from zdohnal/fix_github_actions

workflows.yaml: Fix Github Actions and various fixes

workflows.yaml: Fix Github Actions on i386

- build i386 was failing due missing dependencies - libgcc-s1:i386 and
  libstdc++6:i386
- use ubuntu-latest, so we automatically use the latest stable, but
  print out the pretty name in workflow to show which version it is
  (dont wanna to look up the names :))
- update the image before testing

Merge pull request #709 from nmuggli/media-col-database-custom-size

Add custom page size values to media-col-database

Add custom page size values to media-col-database

When creating the IPP attrs based on the PPD file, if the PPD file
supports custom page sizes update our media-col-database to contain a
media-size attr with the min/max range.