From d4c8d975f3b4e6d64b4478c03f1cbadef60c68aa Mon Sep 17 00:00:00 2001
From: Zdenek Dohnal <zdohnal@redhat.com>
Date: Mon, 6 Sep 2021 13:24:41 +0200
Subject: [PATCH] CUPS Web UI: Fix uninitialized variable during driver
 selection

If device uri doesn't contain model name, `current_make_and_model` is
uninitialized during driver selection in CUPS Web UI.

The issue manifests itself during choosing IPP Everywhere driver, which
has the variable in its name, so it ends up into
`{current_make_and_model} - IPP Everywhere`.

Since we don't have any info about model at the time, I would propose to
just shorten the driver name for such cases. The PR works that way.
---
 CHANGES.md                  | 1 +
 templates/choose-model.tmpl | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGES.md b/CHANGES.md
index a2b393a1d4..4d6bfc6db0 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -64,6 +64,7 @@ CUPS v2.4rc1 (Pending)
   `LPDConfigFile`, `KeepAliveTimeout`, `RIPCache`, and `SMBConfigFile`
   directives in `cupsd.conf` and `cups-files.conf`.
 - Stubbed out deprecated `httpMD5` functions.
+- Fixed uninitialized variable during driver selection in Web UI
 
 
 CUPS v2.3.3op2 (February 1, 2021)
diff --git a/templates/choose-model.tmpl b/templates/choose-model.tmpl
index e916cf8a09..9c9b71f22c 100644
--- a/templates/choose-model.tmpl
+++ b/templates/choose-model.tmpl
@@ -39,7 +39,7 @@
 <TD>
 <SELECT NAME="PPD_NAME" SIZE="10">
 {op=add-printer?:<OPTION VALUE="__no_change__" SELECTED>Current Driver - {current_make_and_model}</OPTION>:}
-{show_ipp_everywhere?<OPTION VALUE="everywhere" SELECTED>{current_make_and_model} - IPP Everywhere &trade;</OPTION>:}
+{show_ipp_everywhere?<OPTION VALUE="everywhere" SELECTED>{current_make_and_model?{current_make_and_model} -:} IPP Everywhere &trade;</OPTION>:}
 {[ppd_name]<OPTION VALUE="{ppd_name}" {op=modify-printer?:{?current_make_and_model={ppd_make_and_model}?SELECTED:}}>{ppd_make_and_model} ({ppd_natural_language})
 }</SELECT>
 </TD>
-- 
2.47.2