From a0340a0995efae59d153eb8126474ef4b6f8320b Mon Sep 17 00:00:00 2001 From: Oliver O'Halloran Date: Thu, 7 Nov 2019 16:58:29 +1100 Subject: libstb/secvar: Ensure secvar_set_status() is called once Remove the check to see if the "status" property already exists. This function is intended to only be called once so failing an assert is fine since it indicates a programming error. Signed-off-by: Oliver O'Halloran --- libstb/secvar/secvar_devtree.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libstb/secvar/secvar_devtree.c b/libstb/secvar/secvar_devtree.c index fcff711..5489db7 100644 --- a/libstb/secvar/secvar_devtree.c +++ b/libstb/secvar/secvar_devtree.c @@ -47,11 +47,8 @@ void secvar_set_status(const char *status) if (!secvar_node) return; // Fail boot? - if (dt_find_property(secvar_node, "status")) - return; - + /* This function should only be called once */ dt_add_property_string(secvar_node, "status", status); - // Fail boot if not successful? } -- cgit v1.1