<div dir="auto">Looks good to me.<div dir="auto"><br></div><div dir="auto">FWIW the one in untar may have to be annotated as ok in the code. They legitimately see a potential race condition but we should document that untar should be allowed to run with no interference.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 3, 2020, 12:56 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dead code identified by Coverity (CID 1456674). The value of ret<br>
at line 358 is always 0.<br>
---<br>
 cpukit/libfs/src/pipe/fifo.c | 2 --<br>
 1 file changed, 2 deletions(-)<br>
<br>
diff --git a/cpukit/libfs/src/pipe/fifo.c b/cpukit/libfs/src/pipe/fifo.c<br>
index 579f118bfd..0a3cbf3d65 100644<br>
--- a/cpukit/libfs/src/pipe/fifo.c<br>
+++ b/cpukit/libfs/src/pipe/fifo.c<br>
@@ -355,8 +355,6 @@ ssize_t pipe_write(<br>
       pipe->waitingWriters ++;<br>
       PIPE_WRITEWAIT(pipe);<br>
       pipe->waitingWriters --;<br>
-      if (ret != 0)<br>
-        goto out_locked;<br>
<br>
       if (pipe->Readers == 0) {<br>
         ret = -EPIPE;<br>
-- <br>
2.17.1<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>