diff options
author | Patrick Georgi <pgeorgi@google.com> | 2019-09-06 13:11:54 +0200 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-09-07 05:50:36 +0000 |
commit | 54eed312a6a95a54ffc7630596962e5c91e7d7e7 (patch) | |
tree | 7cd9b8daaca82dc228e9c452f4af4047912db4b3 | |
parent | 64b7b025b8c6cdb713e6bc5426de242d12282244 (diff) | |
download | chrome-ec-54eed312a6a95a54ffc7630596962e5c91e7d7e7.tar.gz |
util/ectool: check malloc for NULL
Found by Coverity Scan #141760.
BUG=none
BRANCH=none
TEST=none
Change-Id: Icba3268919e8dde937e00172c288b10e2e3f6e87
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1789147
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Auto-Submit: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Commit-Queue: Patrick Georgi <pgeorgi@chromium.org>
-rw-r--r-- | util/ectool.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/util/ectool.c b/util/ectool.c index ac3514800d..fc366918d4 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -6455,6 +6455,8 @@ int cmd_i2c_xfer(int argc, char *argv[]) if (write_len) { write_buf = malloc(write_len); + if (write_buf == NULL) + return -1; for (i = 0; i < write_len; i++) { write_buf[i] = strtol(argv[i], &e, 0); if (e && *e) { |