zone/OpenAssetTools
zone/OpenAssetTools
2
0
Fork 0
mirror of https://github.com/Laupetin/OpenAssetTools.git synced 2025-04-19 15:52:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: make sure creating directories in OutputPathFilesystem cannot lead to a crash

Browse Source
This commit is contained in:
Jan 2025-01-11 13:05:45 +01:00
parent 27c01e0a41
commit b584cd7423
No known key found for this signature in database
GPG Key ID: 44B581F78FF5C57C
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/ObjCommon/SearchPath/OutputPathFilesystem.cpp
View File

@ -1,6 +1,8 @@
#include "OutputPathFilesystem.h" #include "OutputPathFilesystem.h"
#include <format>
#include <fstream> #include <fstream>
#include <iostream>
namespace fs = std::filesystem; namespace fs = std::filesystem;
@ -17,11 +19,21 @@ std::unique_ptr<std::ostream> OutputPathFilesystem::Open(const std::string& file
return nullptr; return nullptr;
const auto containingDirectory = fullNewPath.parent_path(); const auto containingDirectory = fullNewPath.parent_path();
fs::create_directories(containingDirectory);
std::error_code ec;
fs::create_directories(containingDirectory, ec);
if (ec)
{
std::cerr << std::format("Failed to create folder '{}' when try to open file '{}'\n", containingDirectory, fileName);
return nullptr;
}
std::ofstream stream(fullNewPath, std::ios::binary | std::ios::out); std::ofstream stream(fullNewPath, std::ios::binary | std::ios::out);
if (!stream.is_open()) if (!stream.is_open())
{
std::cerr << std::format("Failed to open file '{}'\n", fileName);
return nullptr; return nullptr;
}
return std::make_unique<std::ofstream>(std::move(stream)); return std::make_unique<std::ofstream>(std::move(stream));
} }