mirror of
https://github.com/Fishwaldo/Star64_linux.git
synced 2025-06-06 06:37:59 +00:00
of/unittest: Missing unlocks on error
Static checkers complain that we should unlock before returning. Which is true. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Frank Rowand <frank.rowand@sony.com> Signed-off-by: Rob Herring <robh@kernel.org>
This commit is contained in:
parent
ee320b33b4
commit
8756cd1ded
1 changed files with 6 additions and 2 deletions
|
@ -2210,14 +2210,14 @@ static __init void of_unittest_overlay_high_level(void)
|
|||
unittest(0,
|
||||
"duplicate property '%s' in overlay_base node __symbols__",
|
||||
prop->name);
|
||||
return;
|
||||
goto err_unlock;
|
||||
}
|
||||
ret = __of_add_property_sysfs(of_symbols, prop);
|
||||
if (ret) {
|
||||
unittest(0,
|
||||
"unable to add property '%s' in overlay_base node __symbols__ to sysfs",
|
||||
prop->name);
|
||||
return;
|
||||
goto err_unlock;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -2232,6 +2232,10 @@ static __init void of_unittest_overlay_high_level(void)
|
|||
|
||||
unittest(overlay_data_add(2),
|
||||
"Adding overlay 'overlay_bad_phandle' failed\n");
|
||||
return;
|
||||
|
||||
err_unlock:
|
||||
mutex_unlock(&of_mutex);
|
||||
}
|
||||
|
||||
#else
|
||||
|
|
Loading…
Add table
Reference in a new issue