[OSS-Tools] [PATCH dt-utils 03/12] barebox-state: fix usage of multiple state instances

["Ulrich Ölmann" <u.oelmann@pengutronix.de>]

Let's assume there are two state instances named "blue" (with variable "x") and
"yellow" (with variable "y") then

  $ barebox-state --name blue --get x
  $ barebox-state --name yellow --get y

would both work, whereas only exactly one of the alternative calls

  $ barebox-state --get blue.x
  $ barebox-state --get yellow.y

would succeed. This commit fixes that bug.

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
---
 src/barebox-state.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/barebox-state.c b/src/barebox-state.c
index 3e6c26003462..76d047559a4f 100644
--- a/src/barebox-state.c
+++ b/src/barebox-state.c
@@ -581,9 +581,10 @@ int main(int argc, char *argv[])
 
 			list_for_each_entry(state, &state_list.list, list) {
 				if (strlen(state->name) == statename_len &&
-				    !strncmp(state->name, sg->arg, statename_len))
+				    !strncmp(state->name, sg->arg, statename_len)) {
 					arg = statename_end + 1;
 					break;
+				}
 			}
 		}
 		if (state == &state_list) {
-- 
2.20.1


_______________________________________________
OSS-Tools mailing list
OSS-Tools@pengutronix.de