Attention is currently required from: Jakub Czapiga, Paul Menzel.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52136 )
Change subject: tests/lib/malloc-test: Fix possible memory overrun
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
Sorry, where is the overrun here? These two symbols point to the same address, right?
Please always try to make a clear distinction between actual bugs and false positives from Coverity and don't just blindly do what the program tells you. (I'm okay with this change to silence the warning if you want it, but commit messages should make clear when something is an actual bug and when it's just placating an overzealous tool.)
--
To view, visit
https://review.coreboot.org/c/coreboot/+/52136
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I19f0718c657d565e515157e66367573e08f51254
Gerrit-Change-Number: 52136
Gerrit-PatchSet: 2
Gerrit-Owner: Jakub Czapiga
jacz@semihalf.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Paul Fagerburg
pfagerburg@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Julius Werner
jwerner@chromium.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Attention: Jakub Czapiga
jacz@semihalf.com
Gerrit-Attention: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Tue, 06 Apr 2021 23:52:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment