diff options
| author | Patrick Georgi <pgeorgi@google.com> | 2018-07-26 12:18:41 +0200 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2018-12-06 23:29:31 +0000 |
| commit | 3a4013c22fd2fbb43aaf97f0082b22d93ed5fcfa (patch) | |
| tree | c3b59bb3fcc41a84bf305454d195be556df0dd3b | |
| parent | 0aaef1843cc9160f060722a080a3623b877a3cfb (diff) | |
| download | chrome-ec-3a4013c22fd2fbb43aaf97f0082b22d93ed5fcfa.tar.gz | |
util/cbi-util: Check pointer before using it
Change-Id: If11de8883b001f16d7e8f859a416fbdc5ea0391a
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Found-by: Coverity Scan #187038
Reviewed-on: https://chromium-review.googlesource.com/1151121
Commit-Ready: Patrick Georgi <pgeorgi@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/1355373
Commit-Queue: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/1366309
| -rw-r--r-- | util/cbi-util.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/util/cbi-util.c b/util/cbi-util.c index 5db416fbe4..23d9986348 100644 --- a/util/cbi-util.c +++ b/util/cbi-util.c @@ -333,11 +333,13 @@ static void print_integer(const uint8_t *buf, enum cbi_data_tag tag) { uint32_t v; struct cbi_data *d = cbi_find_tag(buf, tag); - const char *name = d->tag < CBI_TAG_COUNT ? field_name[d->tag] : "???"; + const char *name; if (!d) return; + name = d->tag < CBI_TAG_COUNT ? field_name[d->tag] : "???"; + switch (d->size) { case 1: v = *(uint8_t *)d->value; |