From 3bb08ed694d9c41538988668d8a6ef7874601116 Mon Sep 17 00:00:00 2001 From: Vadim Bendebury Date: Thu, 30 Nov 2017 19:50:03 -0800 Subject: [PATCH] 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 Reviewed-on: https://chromium-review.googlesource.com/802994 Reviewed-by: Mary Ruthven --- extra/usb_updater/gsctool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/extra/usb_updater/gsctool.c b/extra/usb_updater/gsctool.c index ab76199531..a0b4c28c7a 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); }