]> granicus.if.org Git - neomutt/commitdiff
Fix sidebar check_sec.sh warnings.
authorKevin McCarthy <kevin@8t8.us>
Sat, 4 Jun 2016 18:32:04 +0000 (11:32 -0700)
committerRichard Russon <rich@flatcap.org>
Sun, 3 Jul 2016 13:31:41 +0000 (14:31 +0100)
Use safe_malloc, FREE, and the safe_strcat functions.

sidebar.c

index aa2c91d726b79cc883cb40b698e2108786edba4a..6a44260ef510c3972651a277a6119a0da9d8dd83 100644 (file)
--- a/sidebar.c
+++ b/sidebar.c
@@ -783,10 +783,12 @@ static void draw_sidebar (int first_row, int num_rows, int div_width)
         if (option (OPTSIDEBARSHORTPATH))
           tmp_folder_name += lastsep;  /* basename */
         sidebar_folder_name = malloc (strlen (tmp_folder_name) + sidebar_folder_depth*strlen (NONULL(SidebarIndentString)) + 1);
+                                 int sfn_len = strlen (tmp_folder_name) + sidebar_folder_depth*strlen (NONULL(SidebarIndentString)) + 1;
+        sidebar_folder_name = safe_malloc (sfn_len);
         sidebar_folder_name[0]=0;
         for (i=0; i < sidebar_folder_depth; i++)
-          strncat (sidebar_folder_name, NONULL(SidebarIndentString), strlen (NONULL(SidebarIndentString)));
-        strncat (sidebar_folder_name, tmp_folder_name, strlen (tmp_folder_name));
+          safe_strcat (sidebar_folder_name, sfn_len, NONULL(SidebarIndentString));
+        safe_strcat (sidebar_folder_name, sfn_len, tmp_folder_name);
       }
     }
     char str[STRING];