diff options
author | Karthikeyan Ramasubramanian <kramasub@google.com> | 2022-08-08 14:30:03 -0600 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2022-09-04 17:06:21 +0000 |
commit | 0dd3cf4534c4ab9497b5c7b3af5e0ed8b9e919df (patch) | |
tree | 0ddf07cd4cbc75c1f3431d7a793639e5dfdd4e05 /util/scripts | |
parent | 7dd42da9a064750699058b2a9d66d84bc314b221 (diff) |
util/amdfwtool/amdfwread: Fix incorrect option index
index I/O argument to getopt_long is not the index to argv. Instead it
is an index into the optlong array corresponding to the parsed option.
Also getopt() uses a global variable optind to track the index of the
next argument to be processed. Use the optindex variable as an index to
extract the filename from argv.
BUG=None
TEST=Build and use amdfwread to read the Soft-fuse bits from Guybrush
BIOS image. Observed no changes before and after the changes.
Change-Id: I33c74a0c8e12c5af76954524cf7294b7541d286b
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/66553
Reviewed-by: Robert Zieba <robertzieba@google.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/scripts')
0 files changed, 0 insertions, 0 deletions