<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/23473">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">util/lint/checkpatch.pl: Untaint filename<br><br>This fixes the warning that is seen on the jenkins server:<br>Insecure dependency in piped open while running setgid at<br>util/lint/checkpatch.pl line 958.<br><br>Change-Id: I476efa76ef6a275584a47ec0ecf2315948d53e9d<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M util/lint/checkpatch.pl<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/73/23473/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl</span><br><span>index e9a0cc4..c35a0d6 100755</span><br><span>--- a/util/lint/checkpatch.pl</span><br><span>+++ b/util/lint/checkpatch.pl</span><br><span>@@ -951,6 +951,10 @@</span><br><span> my $vname;</span><br><span> for my $filename (@ARGV) {</span><br><span>     my $FILE;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+   # coreboot: Mark filename as untainted</span><br><span style="color: hsl(120, 100%, 40%);">+        $filename =~ /^(.*)$/s or die; $filename = $1;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>     if ($git) {</span><br><span>          open($FILE, '-|', "git format-patch -M --stdout -1 $filename") ||</span><br><span>                  die "$P: $filename: git format-patch failed - $!\n";</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/23473">change 23473</a>. To unsubscribe, or for help writing mail filters, 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/23473"/><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: I476efa76ef6a275584a47ec0ecf2315948d53e9d </div>
<div style="display:none"> Gerrit-Change-Number: 23473 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>