diff options
author | Jonathan Doman <jonathan.doman@intel.com> | 2021-10-19 00:38:45 +0300 |
---|---|---|
committer | Patrick Williams <patrick@stwcx.xyz> | 2021-10-23 00:57:30 +0300 |
commit | 570ebbb8707e9f734f2eef9e37121dc760df640a (patch) | |
tree | 2dffcf0f30ecb95a0c8129575262a3ff7f499e15 /meta-bytedance | |
parent | cacf03a5d50780532a6414c83f347af18b9531c8 (diff) | |
download | openbmc-570ebbb8707e9f734f2eef9e37121dc760df640a.tar.xz |
Fix phosphor-deploy-ssh-keys.bbclass
The logic was broken due to lack of quotes around the semicolon
delimiters inside subshell commands. It did not work for single keys nor
multiple keys provided in SSH_KEYS.
This commit adds proper quotes, and also refactors the loop a bit to
reduce repeated commands, reduce indentation, and clarify variables.
Tested:
- Added INHERIT/SSH_KEYS to local.conf as described in script comments.
- Tested with 1 key: "root:/path/to/key.pub" and 2 keys
"root:/path/to/key.pub;root:/path/to/key2.pub".
- Verified that in all cases keys were added to authorized_keys files
exactly once by checking rootfs output:
(build/tmp/work/<machine>/<image>/<version>/rootfs/home/root/.ssh/authorized_keys)
Signed-off-by: Jonathan Doman <jonathan.doman@intel.com>
Change-Id: I58af4e8107daa6447b8276a66fc7c91e346c7dd5
Diffstat (limited to 'meta-bytedance')
0 files changed, 0 insertions, 0 deletions