<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20682">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/cannonlake: Keep variable from going out of scope<br><br>The variable p was going out of scope while still being pointed to by<br>*cpu_name.<br><br>Fix coverity ID 1378215 (Pointer to local outside scope)<br><br>Change-Id: I6ad7b1919104b4d97869efe5065e39c2a43de638<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/soc/intel/cannonlake/bootblock/report_platform.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/82/20682/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/cannonlake/bootblock/report_platform.c b/src/soc/intel/cannonlake/bootblock/report_platform.c<br>index c508142..7bc93e6 100644<br>--- a/src/soc/intel/cannonlake/bootblock/report_platform.c<br>+++ b/src/soc/intel/cannonlake/bootblock/report_platform.c<br>@@ -68,13 +68,13 @@<br>     msr_t microcode_ver;<br>  static const char * const mode[] = {"NOT ", ""};<br>  const char *cpu_type = "Unknown";<br>+  u32 p[13];<br> <br>         index = 0x80000000;<br>   cpuidr = cpuid(index);<br>        if (cpuidr.eax < 0x80000004) {<br>             strcpy(cpu_string, "Platform info not available");<br>  } else {<br>-             u32 p[13];<br>            int j=0;<br> <br>           for (i = 2; i <= 4; i++) {<br></pre><p>To view, visit <a href="https://review.coreboot.org/20682">change 20682</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/20682"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I6ad7b1919104b4d97869efe5065e39c2a43de638 </div>
<div style="display:none"> Gerrit-Change-Number: 20682 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>