diff options
| author | Patrick Georgi <pgeorgi@google.com> | 2018-08-02 12:28:56 +0200 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-08-15 16:37:11 -0700 |
| commit | fc0f071e171c3eb8904d0eaf2b609a328ce9c7ff (patch) | |
| tree | 51a1a61ca33885c838b165ea5bb9551001d04cf3 /util/ectool_keyscan.c | |
| parent | b7442b335d6b6a8549eb677058ae8f2782a82792 (diff) | |
| download | chrome-ec-fc0f071e171c3eb8904d0eaf2b609a328ce9c7ff.tar.gz | |
driver/accelgyro_lsm6dsm: prevent negative array access
While it "should" never happen, the fifo_next has some corner cases
where it might return a negative index.
The alternative to this approach is to make sure it never underflows, so
we don't need to guard against that.
Change-Id: I86b4e89598329b7a5039a2bd016fc46230b617ac
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Found-by: Coverity Scan #187061
Reviewed-on: https://chromium-review.googlesource.com/1160302
Commit-Ready: Patrick Georgi <pgeorgi@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Stefan Reinauer <reinauer@google.com>
Diffstat (limited to 'util/ectool_keyscan.c')
0 files changed, 0 insertions, 0 deletions