diff options
| author | Subrata Banik <subrata.banik@intel.com> | 2017-07-24 12:00:39 +0530 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2017-07-25 14:59:07 +0000 |
| commit | 815417145e4613716a0e01207e97d0ae245d9d04 (patch) | |
| tree | 444fe371a3c883b0cb58572222e5b3ff2016397d | |
| parent | 4859ce0b815e64e8fbe2cea9407eed83a77a7c6d (diff) | |
| download | coreboot-815417145e4613716a0e01207e97d0ae245d9d04.tar.xz | |
soc/intel/skylake: soc/intel/skylake: 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: Ide53a9267dfba1a00263ada1d7016a48ecb9aad8
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/20739
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
| -rw-r--r-- | src/soc/intel/skylake/me.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/soc/intel/skylake/me.c b/src/soc/intel/skylake/me.c index bf826ede77..4f36c73923 100644 --- a/src/soc/intel/skylake/me.c +++ b/src/soc/intel/skylake/me.c @@ -673,6 +673,7 @@ static int send_heci_reset_message(void) return -1; reply_size = sizeof(reply); + memset(&reply, 0, reply_size); if (recv_heci_message(&reply, &reply_size) == -1) return -1; /* get reply result from HECI MSG */ |