diff options
| author | Simon Glass <[email protected]> | 2026-05-03 14:49:21 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-05-14 15:39:09 -0600 |
| commit | 6b109a1304a03b35ad489e551964737ad80f6a82 (patch) | |
| tree | a37021bad306dedc31fbe01954738ced28905d9c /scripts/const_structs.checkpatch | |
| parent | 825f8ee2fcf64ef5875a0bc0a3e2d8650acdc298 (diff) | |
siemens: Quote A/B flags in env tests
The Siemens am33x-common, env-common and draco-etamin headers gate
boot-partition selection logic on:
if test -n ${A}; then ...
if test -n ${B}; then ...
A and B are flags that the upgrade machinery sets to mark "the other
partition just became active". The default state is unset, in which
case the expression expands to 'test -n' with no operand and relies
on a U-Boot 'test' quirk that treats a missing operand as false to
skip the branch.
Quote each variable so an unset A or B expands to 'test -n ""' and
the emptiness check is explicit.
Fixes: 8b0619579b22 ("cmd: test: fix handling of single-argument form of test")
Signed-off-by: Simon Glass <[email protected]>
Diffstat (limited to 'scripts/const_structs.checkpatch')
0 files changed, 0 insertions, 0 deletions
