Sinipelto
/
banan-os
forked from Bananymous/banan-os
1
0
Fork 0
Code Pull Requests Activity

Kernel: Allow creating directories if path ends with '/' 

Also create and create_dir will now fail with EEXISTS if file exists
This commit is contained in:
Bananymous 2023-12-14 10:56:53 +02:00
parent 6cd5763361
commit e227a87140
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
kernel/kernel/Process.cpp
View File

@ -670,6 +670,9 @@ namespace Kernel
auto parent_inode = TRY(VirtualFileSystem::get().file_from_absolute_path(m_credentials, directory, O_EXEC | O_WRONLY)).inode; auto parent_inode = TRY(VirtualFileSystem::get().file_from_absolute_path(m_credentials, directory, O_EXEC | O_WRONLY)).inode;
if (auto ret = parent_inode->find_inode(file_name); !ret.is_error())
return BAN::Error::from_errno(EEXIST);
if (Inode::Mode(mode).ifdir()) if (Inode::Mode(mode).ifdir())
TRY(parent_inode->create_directory(file_name, mode, m_credentials.euid(), m_credentials.egid())); TRY(parent_inode->create_directory(file_name, mode, m_credentials.euid(), m_credentials.egid()));
else else
@ -792,7 +795,10 @@ namespace Kernel
{ {
LockGuard _(m_lock); LockGuard _(m_lock);
validate_string_access(path); validate_string_access(path);
TRY(create_file_or_dir(path, Inode::Mode::IFDIR | mode)); BAN::StringView path_sv(path);
if (!path_sv.empty() && path_sv.back() == '/')
path_sv = path_sv.substring(0, path_sv.size() - 1);
TRY(create_file_or_dir(path_sv, Inode::Mode::IFDIR | mode));
return 0; return 0;
} }