cpp/openttd-patchpack/source Changeset - r15140:17ee0814362e

Changeset - r15140:17ee0814362e

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

rubidium - 14 years ago 2010-05-10 09:49:02
rubidium@openttd.org

(svn r19779) -Change: add a return type to AppendPathSeparator and use that to determine whether we could append the path separator. If not, do not recurse into that directory as the path would exceed the maximum path length

2 files changed with 10 insertions and 5 deletions:

src/fileio.cpp

src/fileio_func.h

0 comments (0 inline, 0 general)

src/fileio.cpp

➞

Show inline comments

@@ @@ -448,16 +448,21 @@ void FioCreateDirectory(const char *name @@
  * It does not add the path separator to zero-sized strings.
  * @param buf    string to append the separator to
  * @param buflen the length of the buf
  * @return true iff the operation succeeded
  */
-void AppendPathSeparator(char *buf, size_t buflen)
+bool AppendPathSeparator(char *buf, size_t buflen)
+{
 	size_t s = strlen(buf);
 	/* Length of string + path separator + '\0' */
 	if (s != 0 && buf[s - 1] != PATHSEPCHAR && s + 2 < buflen) {
 	if (s != 0 && buf[s - 1] != PATHSEPCHAR) {
 		if (s + 2 >= buflen) return false;
 		buf[s]     = PATHSEPCHAR;
 		buf[s + 1] = '\0';
+	}
 	return true;
+}
 /**
@@ @@ -1109,7 +1114,7 @@ static uint ScanPath(FileScanner *fs, co @@
 			/* Directory */
 			if (!recursive) continue;
 			if (strcmp(d_name, ".") == 0 || strcmp(d_name, "..") == 0) continue;
 			AppendPathSeparator(filename, lengthof(filename));
+			if (!AppendPathSeparator(filename, lengthof(filename))) continue;
 			num += ScanPath(fs, extension, filename, basepath_length, recursive);
 		} else if (S_ISREG(sb.st_mode)) {
 			/* File */
@@ @@ -1196,6 +1201,6 @@ uint FileScanner::Scan(const char *exten @@
+{
 	char path[MAX_PATH];
 	strecpy(path, directory, lastof(path));
 	AppendPathSeparator(path, lengthof(path));
+	if (!AppendPathSeparator(path, lengthof(path))) return 0;
 	return ScanPath(this, extension, path, strlen(path), recursive);
+}

src/fileio_func.h

➞

Show inline comments

@@ @@ -57,7 +57,7 @@ char *FioAppendDirectory(char *buf, size @@
 char *FioGetDirectory(char *buf, size_t buflen, Subdirectory subdir);
 void SanitizeFilename(char *filename);
-void AppendPathSeparator(char *buf, size_t buflen);
+bool AppendPathSeparator(char *buf, size_t buflen);
 void DeterminePaths(const char *exe);
 void *ReadFileToMem(const char *filename, size_t *lenp, size_t maxsize);
 bool FileExists(const char *filename);

0 comments (0 inline, 0 general)