replace strlcpy with strmemccpy

replace applicable calls to strlcpy with calls to strmemccpy
which null terminates on truncation

in theory the strmemccpy calls should be slightly faster since they
don't traverse the rest of the source string on truncation
but I seriously doubt there is too much of that going on in the code base

Change-Id: Ia0251514e36a6242bbf3f03c5e0df123aba60ed2

1 files changed, 5 insertions, 5 deletions

diff --git a/apps/tree.c b/apps/tree.c
index 1939c7ee05..23a909281d 100644
--- a/apps/tree.c
+++ b/apps/tree.c
@@ -383,7 +383,7 @@ static int update_dir(void)
         {
             if (ft_load(&tc, NULL) < 0)
                 return -1;
-            strlcpy(lastdir, tc.currdir, MAX_PATH);
+            strmemccpy(lastdir, tc.currdir, MAX_PATH);
             changed = true;
         }
     }
@@ -573,7 +573,7 @@ void set_current_file(const char *path)
     name = strrchr(path+1,'/');
     if (name)
     {
-        strlcpy(tc.currdir, path, name - path + 1);
+        strmemccpy(tc.currdir, path, name - path + 1);
         name++;
     }
     else
@@ -582,7 +582,7 @@ void set_current_file(const char *path)
         name = path+1;
     }
 
-    strlcpy(lastfile, name, MAX_PATH);
+    strmemccpy(lastfile, name, MAX_PATH);
 
 
     /* If we changed dir we must recalculate the dirlevel
@@ -1007,7 +1007,7 @@ int rockbox_browse(struct browse_context *browse)
             tc.selected_item = 0;
             tc.dirlevel = 0;
 
-            strlcpy(tc.currdir, browse->root, sizeof(tc.currdir));
+            strmemccpy(tc.currdir, browse->root, sizeof(tc.currdir));
         }
 
         start_wps = false;
@@ -1030,7 +1030,7 @@ int rockbox_browse(struct browse_context *browse)
         if (dirfilter != SHOW_ID3DB)
             tc.dirfilter = &global_settings.dirfilter;
         tc.browse = browse;
-        strlcpy(current, browse->root, MAX_PATH);
+        strmemccpy(current, browse->root, MAX_PATH);
         set_current_file(current);
         if (browse->flags&BROWSE_RUNFILE)
             ret_val = ft_enter(&tc);