diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2017-11-30 19:50:03 -0800 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2018-01-02 02:35:17 +0000 |
| commit | 76adb75f9a6d14582bd585baad6d9903a4d5996d (patch) | |
| tree | b778589ce442cd0044e52d78a73074eb740251a8 | |
| parent | 6971941802beafff0b3c675e433eaf10c92b3a16 (diff) | |
| download | chrome-ec-76adb75f9a6d14582bd585baad6d9903a4d5996d.tar.gz | |
gsctool: fix error processing logic
The error processing logic is reversed, which results in missing error
values when errors actually happen.
BRANCH=none
BUG=none
TEST=verified that errors values are now reported properly.
Change-Id: I282920d35e978a704e8c2728a8aa71a5f1da9a00
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/802994
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
(cherry picked from commit 3bb08ed694d9c41538988668d8a6ef7874601116)
Reviewed-on: https://chromium-review.googlesource.com/845439
Reviewed-by: Marco Chen <marcochen@chromium.org>
Commit-Queue: Marco Chen <marcochen@chromium.org>
Tested-by: Marco Chen <marcochen@chromium.org>
| -rw-r--r-- | extra/usb_updater/gsctool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extra/usb_updater/gsctool.c b/extra/usb_updater/gsctool.c index 1c59d180ab..a8cd3cd2c9 100644 --- a/extra/usb_updater/gsctool.c +++ b/extra/usb_updater/gsctool.c @@ -1596,7 +1596,7 @@ static void process_password(struct transfer_descriptor *td) return; fprintf(stderr, "Error setting password: rv %d, response %d\n", - rv, response_size ? 0 : response); + rv, response_size ? response : 0); exit(update_error); } |