<div dir="ltr">I'll pick on you. :)<div><br></div><div>Please don't use the "title" that says "Fix ABC". There has to be a better description.</div><div><br></div><div>Hopefully someone will ack the technical. I think it is right.</div><div><br></div><div>--joel</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 14, 2021 at 9:20 AM Ryan Long <<a href="mailto:ryan.long@oarcorp.com">ryan.long@oarcorp.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">ping<br>
<br>
-----Original Message-----<br>
From: Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank">ryan.long@oarcorp.com</a>> <br>
Sent: Friday, March 12, 2021 8:56 AM<br>
To: <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
Cc: Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank">ryan.long@oarcorp.com</a>><br>
Subject: [PATCH 2/2] main_rtrace.c: Fix Dereference after null check (CID #1399709)<br>
<br>
CID 1399709: Dereference after null check in rtems_trace_buffering_shell_save().<br>
<br>
Closes #4329<br>
---<br>
 cpukit/libmisc/shell/main_rtrace.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/cpukit/libmisc/shell/main_rtrace.c b/cpukit/libmisc/shell/main_rtrace.c<br>
index 753ab9d..e4f59c4 100644<br>
--- a/cpukit/libmisc/shell/main_rtrace.c<br>
+++ b/cpukit/libmisc/shell/main_rtrace.c<br>
@@ -473,6 +473,7 @@ rtems_trace_buffering_shell_save (int argc, char *argv[])<br>
   {<br>
     close (out);<br>
     printf ("error: no memory\n");<br>
+    return 1;<br>
   }<br>
<br>
   memset (buf, 0, SAVE_BUF_SIZE);<br>
--<br>
1.8.3.1<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>