platform/x86: hp-bioscfg: Change how password encoding size is evaluated

Update steps how password encoding size is evaluated

Signed-off-by: Jorge Lopez <jorge.lopez2@hp.com>
Link: https://lore.kernel.org/r/20230731203141.30044-8-jorge.lopez2@hp.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

diff --git a/drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c b/drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c
index afb5190..03d0188 100644 (file)
--- a/drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c
+++ b/drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c
@@ -335,9 +335,12 @@ static int hp_populate_password_elements_from_package(union acpi_object *passwor
                        password_data->max_password_length = int_value;
                        break;
                case PSWD_SIZE:
-                       password_data->encodings_size = int_value;
-                       if (int_value > MAX_ENCODINGS_SIZE)
+
+                       if (int_value > MAX_ENCODINGS_SIZE) {
                                pr_warn("Password Encoding size value exceeded the maximum number of elements supported or data may be malformed\n");
+                               int_value = MAX_ENCODINGS_SIZE;
+                       }
+                       password_data->encodings_size = int_value;
 
                        /* This step is needed to keep the expected
                         * element list pointing to the right obj[elem].type