scheduler/conf.c: Print to stderr if we don't open cups-files.conf

In case cupsd can't open the cups-files.conf, the error message is lost
if journal and syslog don't exist or work on system (usually in
containers).

Log the error into stderr at this place to get the error message if
needed.

Add image/urf and image/pwg-raster filters for test server (for testing AirPrint/IPP Everywhere compatibility.

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

scheduler/job.c: Fix extensive logging in scheduler

Based on currently unknown trigger scheduler sometimes sets JobHistoryUpdate into past, which causes select() to timeout after one second.

It happens when job->file_time of a job without files to remove gets assigned to JobHistoryUpdate. If we check for job->num_files and assign the job->file_time only if there are any, we will fix extensive logging (and unneeded cupsd execution) in various places, e.g. cleaning jobs, expiring subscriptions, deleting temporary queues...

Fixes #604

scheduler/job.c: Fix extensive logging in scheduler

Based on currently unknown trigger scheduler sometimes sets
JobHistoryUpdate into past, which causes `select()` to timeout after one
second.

It happens when `job->file_time` of a job without files to remove gets
assigned to `JobHistoryUpdate`. If we check for `job->num_files` and
assign the `job->file_time` only if there are any, we will fix extensive
logging (and unneeded cupsd execution) in various places, e.g. cleaning
jobs, expiring subscriptions, deleting temporary queues...

Fixes #604

Enable manual builds.

Update LibreSSL version used by VC projects.

Update REPORTING_ISSUES.md

Use `lpstat -l -e` in the document - its output gives easier distinction between temporary and permanent queues.

Thank you Brian Potkin for suggestion!

Merge pull request #745 from zdohnal/ippeveprint_man

Update ippeveprinter documentation

Apply feedback

Update help pages via mantohtml

mantohtml.c: Fix leftovers of strtod()->atof() migration

man: Fix ippeveprinter man page

- explain further parameter `-F` (fixes #744)
- add missing explanation of parameters `-K` and `-a`

Update pt-BR translation (#613)

Translations provided by Guilherme Gonzaga, thanks a lot!

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

Update CHANGES.md

Use "purge-job" instead of "purge-jobs" when canceling a single job (#742)

The command "cancel -x <job>" adds "purge-jobs true" to the Cancel-Job
operation; however, the correct attribute to use for Cancel-job is
"purge-job" (singular), not "purge-jobs" (plural).  As a result, job
files are not removed from /var/spool/cups when "cancel -x <job>" is
executed.

This patch resolves the issue by adding "purge-job" when the IPP
operation is Cancel-Job and "purge-jobs" for other IPP operations
(Purge-Jobs, Cancel-Jobs, and Cancel-My-Jobs)

Update CHANGES.md

Fix delays in printing to lpd when reserved ports are exhausted (Issue #741)

We use our internal implementation for BSD resvport() by default in LPD backend, which should act on port ranges 512 - 1023, however we didn't go back to the highest port number once we got under the lowest port number. This caused a delay 511s long until  `lpd_queue()` got the port number to 0, which triggered the assignment of the highest port number.

The highest port number is assigned right after we cross the lowest port number, which eliminates this kind of delay.

PR by Bryan Mason from Red Hat

Fix delays printing to lpd when reserved ports are exhausted

cups_rresvport() doesn't reserve ports less than 512; however,
lpd_queue() continues decrementing the port number to 0.  This leads
to delays of ~511 seconds once all ports between 512-1023 are
exhausted.  Even when ports become available, lpd_queue() still tries
calling cups_rresvport() with port numbers less than 512, waiting one
second between each call.

Update CHANGES.md

Merge pull request from GHSA-qjgh-5hcq-5f25

Log result of httpGetHostname BEFORE closing the connection

Merge pull request #729 from zdohnal/cupssinglefile

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

Merge pull request #726 from AtariDreams/Fix-regression

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!

Log result of httpGetHostname BEFORE closing the connection

httpClose frees the memory of con->http. This is problematic because httpGetHostname then tries to access the memory it points to.

We have to log the hostname first.

Update CHANGES.md

Merge pull request #721 from zdohnal/cupsgetnameddest_crash

dest.c: Call cupsEnumDests() only if data.name is not NULL (fixes #719)

Fix indentation

dest.c: Call cupsEnumDests() only if data.name is not NULL (fixes #719)

Fixes regression created by #452 - in case there is no default
destination and `cupsGetNamedDest()` is called to get one (by calling it
with argument `name` as NULL), the function crashes.

It happens because we try to look for the default printer on the network
(where we access `data.name`, which is NULL, in callback), but we never
found out the printer's name.

Original fix by Emilio Cobos Alvarez.

Prep CUPS 2.5 development.

Merge pull request #716 from 10ne1/master

config-scripts/cups-common.m4: fix error message typo

config-scripts/cups-common.m4: fix error message typo

$COMPONENT should be $COMPONENTS to properly print
the bad build component, otherwise it's empty.

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.

Merge branch 'master' of https://github.com/l1gi/cups

host_os_name value in config-scripts has changed to 'solaris' some time ago

Merge pull request #461 from zdohnal/color_override_correction

backend/ipp.c: Adjust job options with similar meaning to the same values

backend/ipp.c: Adjust color job options

In case the client doesn't do it, we have to consolidate color options
in the job to prevent opposite options being set at the same time.

Merge pull request #688 from Plombo/media-type-no-duplicate-names

Fix duplicate PPD->IPP media-type name mappings

Fix duplicate PPD->IPP media-type name mappings

If more than one MediaType in a PPD matched the pattern for a registered
PWG media-type name, they would all be mapped to that name, making it
impossible for a client using CUPS' IPP interface to distinguish between
them or specify a particular one.

This comes up with Epson-provided PPDs, which frequently hit this case
by combining print quality with media type like so:
*MediaType PLAIN_HIGH/Plain paper-High: "
*MediaType PLAIN_NORMAL/Plain paper-Standard: "

Solve this problem by not assigning the standard PWG name to either one
when such duplication occurs.

Merge pull request #706 from meshantz/delay-defaults-docs

Document default repeat interval

Document default repeat interval

Potential fix for Kyocera driver bug (Issue #619)

host_os_name value in config-scripts has changed to 'solaris' some time ago

Merge pull request #692 from AtariDreams/inverse-check

Check that the number of trays is consistent before fetching anymore data

Merge pull request #695 from ggodlewski/master

Fix ipptool JSON output

Check that the number of trays is consistent before fetching anymore data

There is no point in getting this data otherwise if the number of sources defined for the media_sources and for the input_tray differ.

Fix ipptool JSON output

Merge pull request #591 from AtariDreams/XOR-3

Use XOR for integers that are always 1 or 0

Use XOR for integers that are always 1 or 0

This is more efficient than using the ! operator on older platforms.

Merge pull request #655 from AtariDreams/What-We-Learned

Free memory leaked by Darwin's Core Foundation

Free memory leaked by Darwin's Core Foundation

There are cases where we do not always free the CF objects we allocate, causing them to leak.

Merge pull request #686 from AtariDreams/print_ptr

Use print_ptr instead of print_buffer

Merge pull request #684 from AtariDreams/wordlen

Wordlen should be a size_t, not an int

Wordlen should be a size_t, not an int

text is always more than or equal to ptr, so it will never underflow.

Use print_ptr instead of print_buffer

It was forgotten about, making the whole bytes addition useless.

Merge pull request #687 from davebarkerxyz/fix-persist-color-mode

Fix check for existing color mode setting

Fix check for existing color mode setting

Colour mode selection is not persisted correctly across restarts. Current check is for "printer-color-mode", but every other reference to this option is "print-color-mode" (no "er"). This was causing the selected colour mode to be reset to the PPD default on restart.

Update copyright.

Remove unused variable.

Fix an integer overflow issue in the PWG media size name code (Issue #668)

Merge pull request #656 from AtariDreams/u

Use %u for unsigned, not signed

Use %u for unsigned, not signed

Merge pull request #590 from AtariDreams/memory

Improve state handling in cases of memory allocation failure

Improve state handling in cases of memory allocation failure

Many lines of code assume that malloc will not fail. In cases where it does, sometimes the program does not know, and as a result, memory can leak and more disastrous consequences can happen before the program ultimately finds something is wrong and then calls exit();

Merge pull request #642 from zdohnal/raise_fortify

configure: Raise FORTIFY_SOURCE level to 3

Drop Codacy code scanning.

Merge pull request #659 from AtariDreams/strncmp

Cast to unsigned long instead of long for size_t

Cast to unsigned long instead of long for size_t

size_t is unsigned, so let's not worry about signedness.

Merge pull request #669 from AtariDreams/fields

UBSan: Array over-read when operating on _fields

configure: Use _FORTIFY_SOURCE if not defined already

If _FORTIFY_SOURCE is not defined in flags, use its level 3.

Backported from libcups.

UBSan: Array over-read when operating on _fields

We are reading outside of the _fields boundaries and onto other fields when we iterate across all the fields, rather than comparing just those that are within _fields, and then always freeing the ones that are not.

This PR fixes that.

Merge pull request #670 from AtariDreams/pos-fp

UBSan: Contain pos - fp->bufpos in parentheses

UBSan: Contain pos - fp->bufpos in parentheses

Yes I know this shouldn't change the behavior in theory, but clang insists it does in practice, so to quiet the UBSan, I put parentheses around pos - fp->bufpos.

Fix coverity-detected typo.

Fix build errors.

Fix a few Coverity-reported issues.

filter/pstops.c: Fix incorrect comparison size for %%PageResources

Fixes #658

Clean up OpenSSL fixes and changelog (Issue #652)

cups/tls-openssl.c: Use general names in certificates (fixes #652)

This results into having only one Subject Alternative Name extension,
which fixes SEC_ERROR_EXTENSION_VALUE_INVALID in Firefox.

REPORTING_ISSUES.md: Make the file more readable

Merge pull request #617 from AtariDreams/cupsMakeServerCredentials

cupsMakeServerCredentials should return 0 on failure, 1 on success