<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Oct 10, 2020, 8:16 AM Sebastian Huber <<a href="mailto:sebh@rtems.org">sebh@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Module:    rtems<br>
Branch:    master<br>
Commit:    bc7ac71f8aaaff0c73469246c526741ec5b181ed<br>
Changeset: <a href="http://git.rtems.org/rtems/commit/?id=bc7ac71f8aaaff0c73469246c526741ec5b181ed" rel="noreferrer noreferrer" target="_blank">http://git.rtems.org/rtems/commit/?id=bc7ac71f8aaaff0c73469246c526741ec5b181ed</a><br>
<br>
Author:    Frank Kühndel <<a href="mailto:frank.kuehndel@embedded-brains.de" target="_blank" rel="noreferrer">frank.kuehndel@embedded-brains.de</a>><br>
Date:      Mon Oct  5 16:37:23 2020 +0200<br>
<br>
libblock: Fix string truncation warning<br>
<br>
This patch does not only fix the compiler warning below. memcpy() is the<br>
better function at this place as the terminating NUL character is never<br>
copied here. Instead more characters will be appended to the<br>
'logical_disk_name' later on.<br>
<br>
../../../cpukit/libblock/src/bdpart-register.c:41:5:<br>
warning: 'strncpy' output truncated before terminating nul copying<br>
as many bytes from a string as its length [-Wstringop-truncation]<br>
<br>
---<br>
<br>
 cpukit/libblock/src/bdpart-register.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/cpukit/libblock/src/bdpart-register.c b/cpukit/libblock/src/bdpart-register.c<br>
index 9956e61..8a1de61 100644<br>
--- a/cpukit/libblock/src/bdpart-register.c<br>
+++ b/cpukit/libblock/src/bdpart-register.c<br>
@@ -38,7 +38,7 @@ static char *create_logical_disk_name( const char *disk_name, char **marker)<br>
   char *logical_disk_name = malloc( disk_name_size + RTEMS_BDPART_NUMBER_SIZE);<br>
<br>
   if (logical_disk_name != NULL) {<br>
-    strncpy( logical_disk_name, disk_name, disk_name_size);<br>
+    memcpy( logical_disk_name, disk_name, disk_name_size);<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Could we have a comment in the code explaining why this is safe?</div><div dir="auto"><br></div><div dir="auto">Please do that for everywhere memcpy is being inserted. Next time someone looks at this code, it will not be obvious. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
     *marker = logical_disk_name + disk_name_size;<br>
   }<br>
<br>
<br>
_______________________________________________<br>
vc mailing list<br>
<a href="mailto:vc@rtems.org" target="_blank" rel="noreferrer">vc@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/vc" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/vc</a></blockquote></div></div></div>