From: Aschref Ben Thabet <aschref.ben-tha...@embedded-brains.de>
strncpy is considered unsafe as the resulting string may not be NULL
terminated. In addition it fills the unused part of the destination
buffer unnecessarily with NULL bytes.
strlcpy is designed to solve the null-termination problems – it always
null-terminates. It’s certainly an improvement over strncpy.
---
cpukit/libmisc/shell/main_edit.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/cpukit/libmisc/shell/main_edit.c b/cpukit/libmisc/shell/main_edit.c
index feefd6bff1..ab2b9eacbf 100644
--- a/cpukit/libmisc/shell/main_edit.c
+++ b/cpukit/libmisc/shell/main_edit.c
@@ -286,8 +286,8 @@ static struct editor *find_editor(struct env *env, char
*filename) {
struct editor *ed = env->current;
struct editor *start = ed;
- if (!realpath(filename, fn)) strncpy(fn, filename, sizeof(fn)-1);
-
+ if (!realpath(filename, fn)) strlcpy(fn, filename, sizeof(fn));
+
do {
if (strcmp(fn, ed->filename) == 0) return ed;
ed = ed->next;
@@ -297,7 +297,7 @@ static struct editor *find_editor(struct env *env, char
*filename) {
static int new_file(struct editor *ed, char *filename) {
if (*filename) {
- strncpy(ed->filename, filename, sizeof(ed->filename)-1);
+ strlcpy(ed->filename, filename, sizeof(ed->filename));
}
else {
sprintf(ed->filename, "Untitled-%d", ++ed->env->untitled);
@@ -1776,8 +1776,8 @@ static void save_editor(struct editor *ed) {
return;
}
}
- strncpy(
- ed->filename, (const char*)ed->env->linebuf, sizeof(ed->filename)-1);
+ strlcpy(
+ ed->filename, (const char*)ed->env->linebuf, sizeof(ed->filename));
ed->newfile = 0;
}
--
2.26.2
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
