diff options
| author | Jett Rink <jettrink@chromium.org> | 2018-08-02 11:14:09 -0600 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2018-12-08 02:14:28 +0000 |
| commit | 0036aa2ef97e676917cdff31e771c614b73f0b32 (patch) | |
| tree | 6022eb7ff80df140575de8eb3e655c323770f6ca | |
| parent | f964335860f66ce188961824e46d7a3ed1e3abc6 (diff) | |
| download | chrome-ec-0036aa2ef97e676917cdff31e771c614b73f0b32.tar.gz | |
ectool: set option flag to 0 when unset
I noticed that subsequent runs of `ectool cbi get 0` after
`ectool cbi get 0 1` would force the EC to read the EEPROM when the
flag wasn't specified. It looks like the previous stack may have been
used.
BRANCH=none
BUG=none
TEST=none
Change-Id: Iba1aae64fb690ee7f3affa632185d53ee63c3c89
Signed-off-by: Jett Rink <jettrink@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1161001
Reviewed-by: Jonathan Brandmeyer <jbrandmeyer@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/1364310
Reviewed-by: YH Lin <yueherngl@chromium.org>
Commit-Queue: YH Lin <yueherngl@chromium.org>
Tested-by: YH Lin <yueherngl@chromium.org>
| -rw-r--r-- | util/ectool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/ectool.c b/util/ectool.c index 1935bc270f..9e6c7673eb 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -6516,7 +6516,7 @@ static int cmd_cbi(int argc, char *argv[]) } if (!strcasecmp(argv[1], "get")) { - struct ec_params_get_cbi p; + struct ec_params_get_cbi p = { 0 }; uint8_t *r; int i; p.tag = tag; |