From: 이형주/Common Platform Lab(SR)/삼성전자 <leee.lee@samsung.com>
Date: Thu, 15 Sep 2022 04:54:47 +0000 (+0900)
Subject: Fix dll filter in doAOTList (#433)
X-Git-Tag: accepted/tizen/unified/20220920.050137^0
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fdotnet%2Flauncher.git;a=commitdiff_plain;h=0c0d4090182aa41feeee9f2b264a3e3c2b2fc712

Fix dll filter in doAOTList (#433)
---

diff --git a/NativeLauncher/tool/ni_common.cc b/NativeLauncher/tool/ni_common.cc
index cecda2a..4796ad3 100644
--- a/NativeLauncher/tool/ni_common.cc
+++ b/NativeLauncher/tool/ni_common.cc
@@ -189,7 +189,7 @@ static bool isTPADll(const std::string& dllPath)
 
 /**
  * @brief create the directory including parents directory, and
- *        copy ownership and smack labels to the created directory.
+ *          copy ownership and smack labels to the created directory.
  * @param[in] target directory path
  * @param[in] source directory path to get ownership and smack label
  * @return if directory created successfully, return true otherwise false
@@ -683,24 +683,21 @@ static ni_error_e doAOTList(std::vector<std::string>& dllList, const std::string
 	std::string coreLib = concatPath(__pm->getRuntimePath(), "System.Private.CoreLib.dll");
 	bool hasSPC = false;
 
-	std::vector<std::string> niList;
 	for (auto it = dllList.begin(); it != dllList.end(); it++) {
 		std::string f = *it;
 		if (!isFile(f)) {
 			_SERR("dll file is not exist : %s", f.c_str());
 			dllList.erase(it--);
 		}
-		if (!isManagedAssembly(f)) {
+		else if (!isManagedAssembly(f)) {
 			_SERR("Input file is not a dll file : %s", f.c_str());
 			dllList.erase(it--);
 		}
 		// handle System.Private.CoreLib.dll separately.
 		// dllList and path manager contain absolute path. So, there is no need to change path to absolute path
-		if (f == coreLib) {
+		else if (f == coreLib) {
 			hasSPC = true;
 			dllList.erase(it--);
-		} else {
-			niList.push_back(changeExtension(f, ".dll", ".ni.dll"));
 		}
 	}