<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 12, 2020 at 7:07 AM Aschref Ben-Thabet <<a href="mailto:aschref.ben-thabet@embedded-brains.de">aschref.ben-thabet@embedded-brains.de</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">From: Aschref Ben Thabet <<a href="mailto:aschref.ben-thabet@embedded-brains.de" target="_blank">aschref.ben-thabet@embedded-brains.de</a>><br>
<br>
replace strncpy with memcpy to silence this warning and free the<br>
allocated memory block.<br></blockquote><div><br></div><div>I don't see a call to strncpy being replaced. Maybe I need coffee. I see an </div><div>RTEMS test assert strcmp.</div><div><br></div><div>Silence what warning?</div><div><br></div><div>I do not  think it is appropriate to replace str*cpy with memcpy. What is the warning?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 testsuites/psxtests/psxndbm01/init.c | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/testsuites/psxtests/psxndbm01/init.c b/testsuites/psxtests/psxndbm01/init.c<br>
index b524aff0df..658af58df3 100644<br>
--- a/testsuites/psxtests/psxndbm01/init.c<br>
+++ b/testsuites/psxtests/psxndbm01/init.c<br>
@@ -216,11 +216,12 @@ rtems_task Init(rtems_task_argument ignored)<br>
   get_phone_no = dbm_fetch( db, name2 );<br>
   rtems_test_assert( strcmp( (const char*)get_phone_no.dptr, PHONE_NO2 ) == 0 );<br>
<br>
-  puts( "Fetch non-existing record and confirm error." );<br>
+  puts( "Fetch non-existing record and confirm error." ); <br></blockquote><div><br></div><div>I don't see a change here. </div><div><br></div><div>And while you are here non-existing isn't a word. It should be "nonexistent"</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
   test_strings = (char*)malloc(6);<br>
   memcpy( test_strings, "Hello", 5 );<br>
<br>
   test_strings[5] = '\0';<br>
+  free(test_strings);<br>
<br>
 /* The data pointed by test_string is now pointed by key.dptr */<br>
   key.dptr = test_strings;<br>
-- <br>
2.26.2<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></div>