Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/20941
Change subject: soc/intel/cannonlake: Initialize struct member to 0 ......................................................................
soc/intel/cannonlake: Initialize struct member to 0
As per GCC 7.1 compiler struct reset_reply is considered as uninitialized inside send_heci_reset_message function.
Change-Id: I01b95d31bfb1d2e9af1704a28dacb9cfd1cdcb50 Signed-off-by: Subrata Banik subrata.banik@intel.com --- M src/soc/intel/cannonlake/reset.c 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/20941/1
diff --git a/src/soc/intel/cannonlake/reset.c b/src/soc/intel/cannonlake/reset.c index 93714d3..0789b17 100644 --- a/src/soc/intel/cannonlake/reset.c +++ b/src/soc/intel/cannonlake/reset.c @@ -65,6 +65,7 @@ return -1;
reply_size = sizeof(reply); + memset(&reply, 0, reply_size); heci_receive(&reply, &reply_size); /* get reply result from HECI MSG */ if (reply.result != 0) {