PPD for driverless IPP: Poll "media-col-database" separately if needed

In the create_local_bg_thread() function for auto-generating a PPD
file for a CUPS queue for a driverless printer, either via the
"everywhere" model selection or an auto-created temporary queue we
need to query the full capabilities information from the printer.

To get the full set of printer properties from a driverless IPP
printer one does a "get-printer-attributes" IPP request with the
attribute "requested-attributes" set to "all,media-col-database" (note
that "all" does not include "media-col-database" because this
attribute is often very long, it contains all valid combinations of
media size, media type, media source, and margins). For some printers
this fails and we fall back to just "all" and lose valuable
information.

But some of those printers which do not support "requested-attributes"
set to "all,media-col-database" support "requested-attributes" set to
"media-col-database" alone and this we now make use of, by polling
"media-col-database" separately and adding it to the IPP response of
"all" if needed.

We discovered such a printer here:

    https://github.com/OpenPrinting/cups-filters/issues/492

Eliminate trivial switch's.

Fix Windows compile error.

Merge pull request #589 from AtariDreams/depth

Remove useless depth argument from static function cups_get_printer_uri

Drop Codacy - serious issues with this code scanning tool and too many false positives and bogus MISRA reports.

Merge pull request #592 from AtariDreams/noreturn

Mark nonreturning functions as _CUPS_NORETURN

Merge pull request #595 from AtariDreams/inheader

Check inheader before checking color space

Merge pull request #597 from AtariDreams/abort

Have fallback in case GetProcAddress fails

Have fallback in case GetProcAddress fails

Merge pull request #580 from mgoppold/master

Log scheme in backend status #579

Check inheader before checking color space

Mark nonreturning functions as _CUPS_NORETURN

Remove useless depth argument from static function cups_get_printer_uri

This is safe to do since this function is not supposed to be exposed via the ABI, nor is there a pointer to this function, so as long as every other function in the file accommodates this change, no behavioral change should happen.

In addition, depth is always 0 and only used for one debug print statement anyway, so we can and should remove it, instead of simply declaring it unused.

Merge branch 'OpenPrinting:master' into master

Keep the original decision for "-offline-report"

Merge pull request #588 from AtariDreams/patch-6

Fix data types on Windows

Fix data types on Windows

Merge pull request #587 from AtariDreams/patch-6

Windows version numbers are unsigned

Windows version numbers are unsigned

Implement ipp-attribute-fidelity in ippeveprinter.

Merge pull request #585 from OpenPrinting/dependabot/github_actions/codacy/codacy-analysis-cli-action-4.2.0

Bump codacy/codacy-analysis-cli-action from 1.1.0 to 4.2.0

Bump codacy/codacy-analysis-cli-action from 1.1.0 to 4.2.0

Bumps [codacy/codacy-analysis-cli-action](https://github.com/codacy/codacy-analysis-cli-action) from 1.1.0 to 4.2.0.
- [Release notes](https://github.com/codacy/codacy-analysis-cli-action/releases)
- [Commits](https://github.com/codacy/codacy-analysis-cli-action/compare/1.1.0...v4.2.0)

---
updated-dependencies:
- dependency-name: codacy/codacy-analysis-cli-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #550 from AtariDreams/canceled

Remove redundant Canceled check

Merge pull request #545 from AtariDreams/clang

Remove dead workaround code

Merge pull request #535 from AtariDreams/arg

Change (argc < 6 || argc > 7) to (argc != 6 && argc != 7)

Merge pull request #462 from jamacku/master

ci: Add Dependabot to keep GitHub actions up to date

Merge pull request #506 from AtariDreams/XPrinter

Correctly display "Xprinter" instead of Xerox for Xprinter devices

Merge pull request #523 from Nopraz/master

Add documentation for compiling on Fedora

Merge pull request #541 from AtariDreams/fix-typo

Fix passing wrong variable to cupsGetOption in testcups

Merge pull request #544 from AtariDreams/final-update

Update rxspenser to latest version

Merge pull request #543 from AtariDreams/Letter-default

Make Letter the default size if the other values do not fit

Add "printer-strings-uri" and "printer-strings-languages-supported" (Issue #551)

Merge pull request #555 from AtariDreams/buffer

Manually assign the two char literals instead of a memcpy

Merge pull request #577 from ferivoz/typos

Fix typos

Merge pull request #578 from ckiee/exp-sub-loglevel

Increase log level for "Expiring subscriptions..."

Transition to current CodeQL tools vs. LGTM.

Allow media-size values to match within +/-1mm (Issue #487)

Drop old badges and references to the non-existent mailing list.

Fix default cupsd keychain with OpenSSL (Issue #529)

Merge pull request #500 from zdohnal/load_ppd_cmyk

scheduler/printers.c: Check for CMYK as well (fixes #421)

Log scheme in backend status

Increase log level for "Expiring subscriptions..."

Fixes https://github.com/NixOS/nixpkgs/issues/195090.

NixOS systems with Avahi enabled would previously spam the journal with
this log message once a second: this hides that spam behind the log level.

Fix typos

Typos found with codespell.

Signed-off-by: Samanta Navarro <ferivoz@riseup.net>

Merge pull request #518 from AtariDreams/admin

Change cupsd.conf to force authentication when accessing administration

Change cupsd.conf to force authentication when accessing administration

With the current default configuration, anyone can request to Find New Printers without being authenticated and with no way to be authenticated. Without authentication cups does not detects printers on the network. The person might not realize that it's because authentication is missing. It seems natural to always offer to authenticate when accessing the administration page.

Co-Authored-By: Dominic Mayers <dominic.mayers@meditationstudies.org>

Fix some cupsd.conf file issues:

- CUPS-Get-Document wasn't password-protected by default.
- The compiled-in default policy wasn't the same as the cupsd.conf default.

Require authentication for CUPS-Get-Document.

Merge pull request #570 from yetamrra/photo-tray-selection

Update InputSlot selection for photo sizes (fixes #569)

Update InputSlot selection for photo sizes (fixes #569)

_ppdCacheGetInputSlot specifically requests the photo tray for paper
sizes up to 5x7" if the client hasn't specified a media-source.  This
can fail if the photo tray is physically smaller than 5x7 (for example,
L-sized).

Instead of explicitly requesting the photo tray, try to request "auto"
and fall back to "photo" if the printer doesn't provide that as an
option.

Signed-off-by: Benjamin Gordon <bmgordon@chromium.org>

Manually assign the two char literals instead of a memcpy

memcpy is just silly here.

Remove redundant Canceled check

We check for it the line below, so let's remove it from the for-loop conditional

Correctly display "Xprinter" instead of Xerox for Xprinter devices

CUPS currently displays "Xerox" instead of "Xprinter." This patch fixes this behavior by changing the comparison to "XPrint ", space included.

Change (argc < 6 || argc > 7) to (argc != 6 && argc != 7)

They both do the same thing, but the latter is more clear.

Fix passing wrong variable to cupsGetOption in testcups

dest is a pointer to a struct, not the array, which is dests.

Honestly, I have no idea how this even compiled in the first place. Yes I know this is testcups, but still.

Make Letter the default size if the other values do not fit

Especially since letter size is the most common size

Update rxspenser to latest version

Remove dead workaround code

Clang no longer thinks newlength can be 0

Merge pull request #567 from AtariDreams/casts

Remove pointless casts

Merge pull request #568 from AtariDreams/auth-ingo

Fix compiler warnings about non-null and casting

Fix compiler warnings about non-null and casting

Explicitly cast to unsigned char, and check for NULL before passing pointers to functions that expect nonnull.

Remove pointless casts

These variables were already in the target type anyway, or implicitly converted to void *.

Fix minor cupsDir API issue on Windows (not setting S_IFREG flag for files)

Merge pull request #565 from AtariDreams/C2

Update config files to latest version

Merge pull request #566 from AtariDreams/strcmp

Change strncmp parameter

Change strncmp parameter

Either the parameter being passed should be 6, not 7, because of the null terminator in the string literal, or strcmp should be used instead.

Update config files to latest version

Bump versions for Windows.

Add unit test for ippDeleteValues.

Merge pull request #557 from sparik/fix-ippdelete

Fix ippDeleteValues corner case.

Fix parens.

Merge pull request #560 from AtariDreams/httpConnect

Replace deprecated httpConnect function with httpConnect2

Replace deprecated httpConnect function with httpConnect2

Fix ippDeleteValues corner case.

Merge pull request #537 from AtariDreams/admin_util

Replace else-if with else

Replace else-if with else

This is just silly otherwise and it is causing unneeded compiler warnings

Merge pull request #538 from AtariDreams/uid_t

Use uid_t to deal with uids instead of integers

Use uid_t to deal with uids instead of integers

We can also use strtoul to convert strings since uid_t is unsigned.

Merge pull request #536 from AtariDreams/atoi

Avoid passing a constant to atoi

Avoid passing a constant to atoi

Use strtol only if server_port is null rather than waste time putting "0" in atoi, which always comes out as 0

Merge pull request #519 from AtariDreams/trivial

Fix trivial deprecations

Fix trivial deprecations

We should use SecTrustCopyCertificateChain and then access the CFArray instead of using the deprecated SecTrustGetCertificateCount

Use xpc_connection_send_message_with_reply_sync instead of xpc_connection_send_message_with_reply.

This also gives us the added benefit of not risking priority inversion, since xpc_connection_send_message_with_reply_sync takes care of that already.

Finally, use RtlGetVersion instead of deprecated GetVersionExA.

Merge branch 'OpenPrinting:master' into master

Merge pull request #530 from AtariDreams/Fixup-Mac

Update Xcode Project

Merge pull request #532 from AtariDreams/Mistake

[Mistake Bug-Fix]: Fix cupsfilter by setting both array elements to -1

[Mistake Bug-Fix]: Fix cupsfilter by setting both array elements to -1

We mistakenly set filterfds[1 - current][0] = -1; twice instead of doing that and setting filterfds[1 - current][1] to -1 as well

Update Xcode Project

Remove dead file references and enable Dead Code Removal.

Fix according to review

Merge pull request #522 from AtariDreams/windows

Update regex to the modified rxspenser repo

Update regex to the modified rxspenser repo

This repo was linked in the original repo as a newer version, with the current version being the original version that abandoned development 8 years ago.

Nothing has been changed outside of that source, which is from upstream.

Add documentation for compiling on Fedora

Merge pull request #520 from AtariDreams/typo

Fix typos

Fix typos

Fix multiple typos that appear in the codebase.

No other changes

Merge pull request #503 from AtariDreams/z

Update toolset to v143

Merge pull request #505 from AtariDreams/GitHub

Update GitHub Actions Checkout API

Update GitHub Actions Checkout API

The v2 API is deprecated. Update that and the setup-msbuild script used

Update toolset to v143

This toolset is the latest version of the MSVC toolchain and should be used

Add conditional for different VS versions up until 2017

Update CHANGES.md

Merge pull request #501 from yetamrra/mediasizes

Add missing registered media sizes to PWG mapping table

Add missing registered media sizes to PWG mapping table

A number of sizes that were added to ipp-registrations.xml after the
initial PWG 5101.1-2002 document are not currently mapped.  When these
sizes are mapped through a generated IPP Everywhere PPD and back to IPP
media size names, they get changed into auto-generated self-describing
names instead of the official registered names.

This change adds all the non-duplicate missing names.  Most of them
don't have official MediaOptions keywords, so the PPD names are mapped
to dimensions.  With the updated mapping table, all of these sizes can
roundtrip correctly.

This also fixes a missing unit and an incorrect name in two existing
mappings for oe photo sizes.

These entries were not added due to being duplicates of existing sizes:

* iso_id-3_88x125mm duplicates iso_b7_88x125mm.
* oe_photo-s10r_10x15in duplicates na_10x15_10x15in.
* prc_3_125x176mm duplicates iso_b6_125x176mm.
* om_postfix_114x229mm duplicates iso_c6c5_114x229mm.
* prc_10_324x458mm duplicates iso_c3_324x458mm.
* prc_5_110x220mm duplicates iso_dl_110x220mm.

Signed-off-by: Benjamin Gordon <bmgordon@chromium.org>

scheduler/printers.c: Check for CMYK as well (fixes #421)

In `load_ppd()` we would set the `print-color-mode` to `monochrome` only if
the device supported color printing and the default PPD choice for
ColorModel was RGB - this broke settings for color printing for models,
which use `CMYK` for ColorModel.

With this patch we check for CMYK as well.

CUPS 3.0-