diff options
author | Mike Frysinger <vapier@chromium.org> | 2019-08-28 18:38:31 -0400 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-08-29 02:11:38 +0000 |
commit | 31bda282648dd662b1f9c7c8cb8c2dfc8ca27585 (patch) | |
tree | 8ef78f65193ae38d747af21d7dc97c7b4e0c233d /firmware/lib/cgptlib/include/crc32.h | |
parent | afd52397565311166a97f7681209c2f9d3a2d37a (diff) | |
download | vboot-stabilize-12464.B.tar.gz |
cgpt: show: add some sanity checking to -i flagsstabilize-12464.B
If people use -i0, the code runs as if the flag wasn't specified.
Since valid partition numbers are [1,128], and the lower layers
already reject values outside that range (except for 0), add an
explicit check to the CLI for -i0.
Trying to display specific fields w/out -i makes no sense, and
the lower layers just ignore it. Add an explicit check for it
so users don't try to do `cgpt show /dev/sda -s` and wonder why
the output is unchanged.
Passing more than one specific field selector like -s -b doesn't
work -- whatever flag was specified last wins. This isn't that
obvious to users, so throw an explicit error when it happens.
BUG=None
TEST=CQ passes
BRANCH=None
Change-Id: I7c98822b79b389824b544b128ede93458b678342
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/1773964
Reviewed-by: Julius Werner <jwerner@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Diffstat (limited to 'firmware/lib/cgptlib/include/crc32.h')
0 files changed, 0 insertions, 0 deletions