Check only the first byte instead of scanning the entire string with
strnlen().

Signed-off-by: Thorsten Blum <[email protected]>
---
 arch/powerpc/platforms/pseries/papr_platform_attributes.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/papr_platform_attributes.c 
b/arch/powerpc/platforms/pseries/papr_platform_attributes.c
index c6159870de0e..ff8adda02341 100644
--- a/arch/powerpc/platforms/pseries/papr_platform_attributes.c
+++ b/arch/powerpc/platforms/pseries/papr_platform_attributes.c
@@ -325,12 +325,8 @@ static int __init papr_init(void)
        }
 
        for (idx = 0; idx < num_attrs; idx++) {
-               bool show_val_desc = true;
-
                /* Do not add the value desc attr if it does not exist */
-               if (strnlen(esi_attrs[idx].value_desc,
-                           sizeof(esi_attrs[idx].value_desc)) == 0)
-                       show_val_desc = false;
+               bool show_val_desc = *esi_attrs[idx].value_desc != '\0';
 
                if (add_attr_group(be64_to_cpu(esi_attrs[idx].id),
                                   &papr_groups[idx],

Reply via email to