diff options
author | Milton D. Miller II <miltonm@us.ibm.com> | 2016-03-07 17:46:28 -0600 |
---|---|---|
committer | Milton D. Miller II <miltonm@us.ibm.com> | 2016-03-07 17:46:28 -0600 |
commit | a81cb9305e55dbcba81dac585ccdc4d086a1a9fe (patch) | |
tree | 61175ff42f8ee1f26dd62e4cdbbeff06fdded627 /meta-phosphor/common/recipes-phosphor/dbus | |
parent | b8c5174dd737ac0c9ba46585bd8de997254290f3 (diff) | |
download | talos-openbmc-a81cb9305e55dbcba81dac585ccdc4d086a1a9fe.tar.gz talos-openbmc-a81cb9305e55dbcba81dac585ccdc4d086a1a9fe.zip |
init: Increase fidelity of u-boot environment parsing
Deleting a variable with a long value exposed a duplicate
openbmcinitdownloadurl variable resulting in two lines in the
url file, and wget was not happy.
The u-boot environment ends with a double NUL character like
many operating environments. Using strings to separate on the
NUL bytes was loosing this information.
The fw_printenv command does not clear the remainder of the
environment when deleting variables. Instead it just makes sure
it is terminated with a double NUL byte.
Switch from strings to tr to separate the strings. Translate NL
to CR to avoid false matches, and use sed to detect a blank line.
Also use tail to skip over the CRC bytes. We don't have the
config file to know if a flag exists so assume 1 copy for now.
Signed-off-by: Milton Miller <miltonm@us.ibm.com>
Diffstat (limited to 'meta-phosphor/common/recipes-phosphor/dbus')
0 files changed, 0 insertions, 0 deletions