cid#562978 silence Out-of-bounds access
coverity only takes the tag when it sits on the line directly above the statement, and the rest of the tree spells out the FALSE classification Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com> Change-Id: I2cb97c3314a7b612b8fe5bbe2270c7436d1d0717 Reviewed-on: https://gerrit.collaboraoffice.com/c/online/+/10507 Tested-by: Jenkins CPCI <releng@collaboraoffice.com> Reviewed-by: Noel Grandin <noel.grandin@collabora.com> (cherry picked from commit 260f71263c7a37290d50a182ab1e8616a4b99de3) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/210288 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
This commit is contained in:
committed by
Xisco Fauli
parent
57e1ddf8c0
commit
8568a3191d
@@ -103,8 +103,7 @@ static oslSecurityImpl * growSecurityImpl(
|
||||
}
|
||||
p = static_cast<oslSecurityImpl *>(realloc(impl, n));
|
||||
if (p != nullptr) {
|
||||
// coverity[overrun-buffer-arg] - theoretically massive n is not due to
|
||||
// a negative parameter being interpreted as unsigned
|
||||
// coverity[overrun-buffer-arg : FALSE] - n is the size just passed to realloc
|
||||
memset (p, 0, n);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user