Skip to content

Commit

Permalink
libstb/secvar: Ensure secvar_set_status() is called once
Browse files Browse the repository at this point in the history
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 <oohall@gmail.com>
  • Loading branch information
oohal committed Nov 7, 2019
1 parent f9eb53d commit a0340a0
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions libstb/secvar/secvar_devtree.c
Original file line number Diff line number Diff line change
Expand Up @@ -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?
}


Expand Down

0 comments on commit a0340a0

Please sign in to comment.