diff options
author | Vadim Bendebury <vbendeb@chromium.org> | 2018-02-01 11:00:40 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-02-01 18:08:45 -0800 |
commit | 8dba841b4e4e514204f6e002115862fe3229666f (patch) | |
tree | 469598ab8717cfe61ef4ac0029778a8024af8446 | |
parent | 69b592426c7d2e89f9e78025b713ab7f2ce89807 (diff) | |
download | chrome-ec-8dba841b4e4e514204f6e002115862fe3229666f.tar.gz |
ccd: fix scan-admin warnings
Automated code scanner highlighted a few problems in the recent ode
additions. This patch fixes the problems.
BRANCH=cr50
BUG=none
TEST=none
Change-Id: I1f199eb5d2af992384ab04f3010b4b646464a70f
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/897993
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
-rw-r--r-- | common/ccd_config.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/common/ccd_config.c b/common/ccd_config.c index 544ff33454..3e8165e97f 100644 --- a/common/ccd_config.c +++ b/common/ccd_config.c @@ -878,7 +878,8 @@ static int ccd_command_wrapper(int argc, char *password, * error code is the first byte after the header. */ return_code = be32toh(vch->tpm_header.command_code); - if (return_code && (return_code != VENDOR_RC_IN_PROGRESS)) { + if ((return_code != EC_SUCCESS) && + (return_code != VENDOR_RC_IN_PROGRESS)) { rv = vch->ccd_subcommand; } else { rv = EC_SUCCESS; @@ -1457,7 +1458,7 @@ static enum vendor_cmd_rc ccd_vendor(enum vendor_cmd_cc code, size_t input_size, size_t *response_size) { - enum vendor_cmd_rc (*handler)(void *x, size_t y, size_t *t); + enum vendor_cmd_rc (*handler)(void *x, size_t y, size_t *t) = NULL; char *buffer; enum vendor_cmd_rc rc; @@ -1523,7 +1524,7 @@ static enum vendor_cmd_rc ccd_disable_rma(enum vendor_cmd_cc code, size_t input_size, size_t *response_size) { - int rv; + int rv = EC_SUCCESS; int error_line; do { @@ -1555,6 +1556,7 @@ static enum vendor_cmd_rc ccd_disable_rma(enum vendor_cmd_cc code, CPRINTF("Capability %d is not present\n", required_capabilities[i]); error_line = __LINE__; + rv = EC_ERROR_ACCESS_DENIED; break; } } |