[coreboot-gerrit] Change in coreboot[master]: soc/intel/braswell: Fix Coverity Error

Hannah Williams (Code Review) gerrit at coreboot.org
Thu Aug 3 08:11:30 CEST 2017


Hannah Williams has uploaded this change for review. ( https://review.coreboot.org/20870


Change subject: soc/intel/braswell: Fix Coverity Error
......................................................................

soc/intel/braswell: Fix Coverity Error

Change-Id: I286658a6a4c6832fbd6395440cd5383dc81d3c15
Signed-off-by: Hannah Williams <hannah.williams at intel.com>
---
M src/soc/intel/braswell/acpi.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/20870/1

diff --git a/src/soc/intel/braswell/acpi.c b/src/soc/intel/braswell/acpi.c
index 7a51816..429a2b7 100644
--- a/src/soc/intel/braswell/acpi.c
+++ b/src/soc/intel/braswell/acpi.c
@@ -570,7 +570,7 @@
 	/* FIXME if IGD is disabled, we should exit here. */
 
 	memcpy(&opregion->header.signature, IGD_OPREGION_SIGNATURE,
-		sizeof(IGD_OPREGION_SIGNATURE));
+		sizeof(IGD_OPREGION_SIGNATURE) - 1);
 
 	/* 8kb */
 	opregion->header.size = sizeof(igd_opregion_t) / 1024;

-- 
To view, visit https://review.coreboot.org/20870
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I286658a6a4c6832fbd6395440cd5383dc81d3c15
Gerrit-Change-Number: 20870
Gerrit-PatchSet: 1
Gerrit-Owner: Hannah Williams <hannah.williams at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170803/b7a84cb2/attachment.html>


More information about the coreboot-gerrit mailing list