LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 01:37:39 +03:00
Code Issues Packages Projects Releases Wiki Activity

ProcFS: Check for empty Optional in read_bytes()

Browse Source 
In read_bytes() we now check that the Optional "data" actually
contains a value before use, avoiding a failing asserting in
kernel space.
This commit is contained in:
Drew Stratford 2019-10-22 22:45:48 +13:00 committed by Andreas Kling
parent 4d99856f95
commit d063734f69
Notes: sideshowbarker 2024-07-19 11:34:50 +09:00 
Author: https://github.com/DrewStratford
Commit: https://github.com/SerenityOS/serenity/commit/d063734f699
Pull-request: https://github.com/SerenityOS/serenity/pull/681
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Kernel/FileSystem/ProcFS.cpp
View File

@ -997,10 +997,14 @@ ssize_t ProcFSInode::read_bytes(off_t offset, ssize_t count, u8* buffer, FileDes
}
auto& data = generated_data;
ssize_t nread = min(static_cast<off_t>(data.value().size() - offset), static_cast<off_t>(count));
memcpy(buffer, data.value().data() + offset, nread);
if (nread == 0 && description && description->generator_cache())
description->generator_cache().clear();
ssize_t nread = 0;
if (data.has_value()) {
nread = min(static_cast<off_t>(data.value().size() - offset), static_cast<off_t>(count));
memcpy(buffer, data.value().data() + offset, nread);
if (nread == 0 && description && description->generator_cache())
description->generator_cache().clear();
}
return nread;
}