summaryrefslogtreecommitdiff
path: root/src/soc/intel/braswell/scc.c
diff options
context:
space:
mode:
authorNico Huber <nico.huber@secunet.com>2021-06-22 13:49:44 +0000
committerWerner Zeh <werner.zeh@siemens.com>2021-06-25 04:28:36 +0000
commitf22f408956bf02609a96b7d72fb3321da159bfc6 (patch)
tree9672a614ab3f9dd4433c95b6bae4ce7d92cc72d4 /src/soc/intel/braswell/scc.c
parentcd85aac434fc50da383b49a17d010a755741d4fa (diff)
cbfstool: Make use of spurious null-termination
The null-termination of `filetypes` was added after the code was written, obviously resulting in NULL dereferences. As some more code has grown around the termination, it's hard to revert the regression, so let's update the code that still used the array length. This fixes commit 7f5f9331d1 (util/cbfstool: fix buffer over-read) which actually did fix something, but only one path while it broke two others. We should be careful with fixes, they can always break something else. Especially when a dumb tool triggered the patching it seems likely that fewer people looked into related code. Change-Id: If2ece1f5ad62952ed2e57769702e318ba5468f0c Signed-off-by: Nico Huber <nico.huber@secunet.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/55763 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Julius Werner <jwerner@chromium.org>
Diffstat (limited to 'src/soc/intel/braswell/scc.c')
0 files changed, 0 insertions, 0 deletions