<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>