<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22900">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">util/cbfstool: Check for NULL before dereference<br><br>Fixed coverity issue: 1302455 - Dereference null return value<br><br>Change-Id: I59b908adc4d35f08fda8e4ad3f806714f2caeb65<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M util/cbfstool/partitioned_file.c<br>1 file changed, 4 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/22900/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/util/cbfstool/partitioned_file.c b/util/cbfstool/partitioned_file.c<br>index 7b4b003..9abd32a 100644<br>--- a/util/cbfstool/partitioned_file.c<br>+++ b/util/cbfstool/partitioned_file.c<br>@@ -196,6 +196,10 @@<br> <br>      const struct fmap_area *fmap_fmap_entry =<br>                             fmap_find_area(file->fmap, SECTION_NAME_FMAP);<br>+<br>+ if (!fmap_fmap_entry)<br>+                return NULL;<br>+<br>       if ((long)fmap_fmap_entry->offset != fmap_region_offset) {<br>                 ERROR("FMAP's '%s' section doesn't point back to FMAP start (did something corrupt this file?)\n",<br>                                                  SECTION_NAME_FMAP);<br></pre><p>To view, visit <a href="https://review.coreboot.org/22900">change 22900</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/22900"/><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: I59b908adc4d35f08fda8e4ad3f806714f2caeb65 </div>
<div style="display:none"> Gerrit-Change-Number: 22900 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>