Allow other than /Yes and /Off in a check box (fixes #1056)

1 files changed, 4 insertions, 4 deletions

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 "