<p>Ryan Salsamendi has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20452">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/x86/name: Fix undefined behavior<br><br>Fixes report found by undefined behavior sanitizer. Dereferencing a<br>pointer that's not aligned to the size of access is undefined behavior.<br><br>Change-Id: I1362a3eb8c97f5c7e848d75f8d1a219968a7ef9e<br>Signed-off-by: Ryan Salsamendi <rsalsamendi@hotmail.com><br>---<br>M src/cpu/x86/name/name.c<br>1 file changed, 3 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/52/20452/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/cpu/x86/name/name.c b/src/cpu/x86/name/name.c<br>index b59fd8b..aa1ba7a 100644<br>--- a/src/cpu/x86/name/name.c<br>+++ b/src/cpu/x86/name/name.c<br>@@ -21,9 +21,8 @@<br> void fill_processor_name(char *processor_name)<br> {<br>   struct cpuid_result regs;<br>-    char temp_processor_name[49];<br>         char *processor_name_start;<br>-  unsigned int *name_as_ints = (unsigned int *)temp_processor_name;<br>+    unsigned int name_as_ints[13];<br>        int i;<br> <br>     for (i = 0; i < 3; i++) {<br>@@ -34,10 +33,10 @@<br>             name_as_ints[i * 4 + 3] = regs.edx;<br>   }<br> <br>- temp_processor_name[48] = 0;<br>+ name_as_ints[12] = 0;<br> <br>      /* Skip leading spaces. */<br>-   processor_name_start = temp_processor_name;<br>+  processor_name_start = (char *)name_as_ints;<br>  while (*processor_name_start == ' ')<br>          processor_name_start++;<br> <br></pre><p>To view, visit <a href="https://review.coreboot.org/20452">change 20452</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/20452"/><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: I1362a3eb8c97f5c7e848d75f8d1a219968a7ef9e </div>
<div style="display:none"> Gerrit-Change-Number: 20452 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Ryan Salsamendi <rsalsamendi@hotmail.com> </div>