diff options
| author | Paul Moore <pmoore2@cisco.com> | 2020-08-24 13:17:58 -0400 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2021-03-10 18:34:37 -0500 |
| commit | 018b74d2d69ef35b43b79709f2ea60325f12dde2 (patch) | |
| tree | d2dbf7233ae49f5972a0edf445db5fe38d84ac3c /Cryptlib/OpenSSL/crypto/x509v3/pcy_node.c | |
| parent | 4033d1fd0f25196f9a35e944679676277cd51960 (diff) | |
| download | efi-boot-shim-018b74d2d69ef35b43b79709f2ea60325f12dde2.tar.gz efi-boot-shim-018b74d2d69ef35b43b79709f2ea60325f12dde2.zip | |
shim: attempt to improve the argument handling
"So, load options are a giant pain in the ass." - Peter Jones
This patch attempts to workaround two LoadOption problems seen on
my test system: arguments separated with only whitespace (L" ") and
strings that aren't properly NULL terminated.
In addition to my test system, it appears at least one other person
has run into a similar problem and I can reproduce the whitespace
delimiter issue with QEMU+OVMF (OEMU v5.2.0, OVMF v202011).
* https://github.com/rhboot/shim/issues/181
For reference, using QEMU+OVMF and a simple test application,
"test.efi", which does a hexdump of LoadOptions I see the following
when run via the UEFI shell:
FS0:\> test.efi one two three
LoadOptions:
0000: 74 00 65 00 73 00 74 00 2E 00 65 00 66 00 69 00 t.e.s.t...e.f.i.
0016: 20 00 6F 00 6E 00 65 00 20 00 74 00 77 00 6F 00 ..o.n.e...t.w.o.
0032: 20 00 74 00 68 00 72 00 65 00 65 00 00 00 ..t.h.r.e.e...
Signed-off-by: Paul Moore <pmoore2@cisco.com>
Diffstat (limited to 'Cryptlib/OpenSSL/crypto/x509v3/pcy_node.c')
0 files changed, 0 insertions, 0 deletions