Fix missing closedir() in Find_File::Find_Internal()

Tasssadar · Gerrit Code Review · commit cbe764c42b7f · 2014-05-31T14:22:07.000+02:00
Change-Id: Ie74570d3203be347390ca74bd00429bde87d6741
Signed-off-by: Vojtech Bocek &lt;vbocek@gmail.com&gt;
diff --git a/find_file.cpp b/find_file.cpp
@@ -33,7 +33,7 @@ Find_File::Find_File() {
 }
 
 string Find_File::Find_Internal(const string& filename, const string& starting_path) {
-	DIR *d = opendir(starting_path.c_str());
+	DIR *d;
 	string new_path, return_path;
 	vector<string> dirs;
 	vector<string> symlinks;
@@ -45,6 +45,7 @@ string Find_File::Find_Internal(const string& filename, const string& starting_p
 	}
 	searched_dirs.push_back(starting_path);
 
+	d = opendir(starting_path.c_str());
 	if (d == NULL) {
 		LOGERR("Find_File: Error opening '%s'\n", starting_path.c_str());
 		return "";

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ Find_File::Find_File() {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`string Find_File::Find_Internal(const string& filename, const string& starting_path) {`
`36`		`- DIR *d = opendir(starting_path.c_str());`
	`36`	`+ DIR *d;`
`37`	`37`	`string new_path, return_path;`
`38`	`38`	`vector<string> dirs;`
`39`	`39`	`vector<string> symlinks;`
`@@ -45,6 +45,7 @@ string Find_File::Find_Internal(const string& filename, const string& starting_p`
`45`	`45`	`}`
`46`	`46`	`searched_dirs.push_back(starting_path);`
`47`	`47`
	`48`	`+ d = opendir(starting_path.c_str());`
`48`	`49`	`if (d == NULL) {`
`49`	`50`	`LOGERR("Find_File: Error opening '%s'\n", starting_path.c_str());`
`50`	`51`	`return "";`