From f3c1a6b19c1ee1b5640348d846ae1ee2898cfb72 Mon Sep 17 00:00:00 2001 From: Scott Lahteine Date: Thu, 16 Jul 2015 12:10:38 -0700 Subject: [PATCH] Reduce lsDive stack usage to the minimum --- Marlin/cardreader.cpp | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/Marlin/cardreader.cpp b/Marlin/cardreader.cpp index b8a940f4b..7e4f3ffe3 100644 --- a/Marlin/cardreader.cpp +++ b/Marlin/cardreader.cpp @@ -56,22 +56,28 @@ void CardReader::lsDive(const char *prepend, SdFile parent, const char * const m // If the entry is a directory and the action is LS_SerialPrint if (DIR_IS_SUBDIR(&p) && lsAction != LS_Count && lsAction != LS_GetFilename) { - // Allocate enough stack space for the full path to a folder, trailing slash, and nul - int len = strlen(prepend) + FILENAME_LENGTH + 1 + 1; - char path[len]; - // Get the short name for the item, which we know is a folder char lfilename[FILENAME_LENGTH]; createFilename(lfilename, p); + // Allocate enough stack space for the full path to a folder, trailing slash, and nul + boolean prepend_is_empty = (prepend[0] == '\0'); + int len = strlen(prepend) + (prepend_is_empty ? 1 : 0) + strlen(lfilename) + 1; + char path[len]; + // Append the FOLDERNAME12/ to the passed string. // It contains the full path to the "parent" argument. // We now have the full path to the item in this folder. - path[0] = '\0'; - if (prepend[0] == '\0') strcat(path, "/"); // a root slash if prepend is empty - strcat(path, prepend); - strcat(path, lfilename); - strcat(path, "/"); + if (prepend_is_empty) { + path[0] = '/'; // a root slash if prepend is empty + path[1] = '\0'; + } + else + path[0] = '\0'; + + strcat(path, prepend); // 1 character minimum + strcat(path, lfilename); // FILENAME_LENGTH-1 characters maximum + strcat(path, "/"); // 1 character // Serial.print(path);