Merge pull request #31 from intel-innersource/p2c/fix-coverity

coverity fixes

Author: markgalvan-intel

IntelPresentMon/AppCef/source/util/async/BrowseReadSpec.h

@@ -40,7 +40,7 @@ namespace p2c::client::util::async
             };
 
             std::wstring payload;
-            if (GetOpenFileNameW(&ofn) && wcsnlen_s(pathBuffer, sizeof(pathBuffer)) > 0) {
+            if (GetOpenFileNameW(&ofn) && wcsnlen_s(pathBuffer, std::size(pathBuffer)) > 0) {
                 std::wifstream file{ pathBuffer };
                 payload = std::wstring(
                     std::istreambuf_iterator<wchar_t>{ file },

IntelPresentMon/AppCef/source/util/async/BrowseStoreSpec.h

@@ -43,7 +43,7 @@ namespace p2c::client::util::async
             bool failed = false;
 
             // try to write the file to disk
-            if (confirmed && wcsnlen_s(pathBuffer, sizeof(pathBuffer)) > 0) {
+            if (confirmed && wcsnlen_s(pathBuffer, std::size(pathBuffer)) > 0) {
                 std::wofstream file{ pathBuffer };
                 file << pArgObj->GetDictionary()->GetString("payload").ToWString();
                 failed = !file.good();

IntelPresentMon/Core/source/infra/util/FolderResolver.cpp

@@ -70,7 +70,7 @@ namespace p2c::infra::util
 		}
 		else
 		{
-			appPath = std::filesystem::current_path().wstring();
+			docPath = std::filesystem::current_path().wstring();
 		}
 
 		// TODO: this really doesn't belong here, but here it stays until time for something saner