From 86ea2012bd413e728c2a3eecf5f2a6de113e3679 Mon Sep 17 00:00:00 2001 From: Jay Berkenbilt Date: Fri, 22 Dec 2023 14:42:16 -0500 Subject: Allow other than /Yes and /Off in a check box (fixes #1056) --- libqpdf/QPDFFormFieldObjectHelper.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'libqpdf/QPDFFormFieldObjectHelper.cc') diff --git a/libqpdf/QPDFFormFieldObjectHelper.cc b/libqpdf/QPDFFormFieldObjectHelper.cc index 4f7ca0e1..371ed271 100644 --- a/libqpdf/QPDFFormFieldObjectHelper.cc +++ b/libqpdf/QPDFFormFieldObjectHelper.cc @@ -304,10 +304,10 @@ QPDFFormFieldObjectHelper::setV(QPDFObjectHandle value, bool need_appearances) bool okay = false; if (value.isName()) { std::string name = value.getName(); - if ((name == "/Yes") || (name == "/Off")) { - okay = true; - setCheckBoxValue((name == "/Yes")); - } + okay = true; + // Accept any value other than /Off to mean checked. Files have been seen that use + // /1 or other values. + setCheckBoxValue((name != "/Off")); } if (!okay) { this->oh.warnIfPossible("ignoring attempt to set a checkbox field to a value of " -- cgit v1.2.3-54-g00ecf