diff options
author | Tom Hughes <tomhughes@chromium.org> | 2019-06-10 15:45:27 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-06-24 21:12:44 +0000 |
commit | b01ef0473c54b65a170d83f9184bfb7eb9583276 (patch) | |
tree | 79cf755ab1d9de820d80147bce733ae114cfd277 /common/fpsensor | |
parent | d28c0a87edee59d458b847fc5455b1c5cd698f15 (diff) | |
download | chrome-ec-b01ef0473c54b65a170d83f9184bfb7eb9583276.tar.gz |
fpsensor: Return correct error code for console command
Console commands return different error code types than host commands.
When the host command return value is returned for a console command,
the error message that is printed on the console is wrong, which is
confusing.
BRANCH=none
BUG=chromium:978078
TEST="syslock" in hatch FP console so system_is_locked returns true
"fpcapture" in hatch FP console => Access Denied
"fpenroll" in hatch FP console => Access Denied
Change-Id: I18dbc093011743d4a9469eb5843c1464496bdd53
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1674049
Reviewed-by: Nicolas Norvez <norvez@chromium.org>
Diffstat (limited to 'common/fpsensor')
-rw-r--r-- | common/fpsensor/fpsensor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/common/fpsensor/fpsensor.c b/common/fpsensor/fpsensor.c index 1e384e5a5f..895f65d8da 100644 --- a/common/fpsensor/fpsensor.c +++ b/common/fpsensor/fpsensor.c @@ -628,7 +628,7 @@ int command_fpcapture(int argc, char **argv) enum ec_error_list rc; if (system_is_locked()) - return EC_RES_ACCESS_DENIED; + return EC_ERROR_ACCESS_DENIED; if (argc >= 2) { char *e; @@ -657,7 +657,7 @@ int command_fpenroll(int argc, char **argv) "Immobile", "Low Coverage"}; if (system_is_locked()) - return EC_RES_ACCESS_DENIED; + return EC_ERROR_ACCESS_DENIED; do { int tries = 1000; |