diff options
author | Sam Atkins <atkinssj@serenityos.org> | 2022-01-17 17:36:20 +0000 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2022-01-20 10:38:10 +0100 |
commit | 94f2508519f7dd221b8db356c41e5f3c698f9774 (patch) | |
tree | 2d749c5f6945a6943a1637db4a1e99d5092ae510 /Userland/Libraries/LibFileSystemAccessClient/Client.h | |
parent | 166221373755503134716317a51762a5fbedf3a7 (diff) | |
download | serenity-94f2508519f7dd221b8db356c41e5f3c698f9774.zip |
CrashReporter: Don't crash when investigating a HackStudio crash
Previously when opening a crash report for HackStudio, the
`unveil("/bin/HackStudio", "rx")` call was failing because of the
earlier `unveil(executable_path.characters(), "r")` call requesting only
"r" permissions for it. This patch handles this specific case, so you
can crash HackStudio to your heart's content. :^)
Also, we were unveiling the executable path twice, once manually and
once implicitly as part of the coredump's libraries, so we now check for
the latter and avoid it.
Thanks to Daniel for noticing what was right in front of me and I didn't
see!
Co-authored-by: Daniel Bertalan <dani@danielbertalan.dev>
Diffstat (limited to 'Userland/Libraries/LibFileSystemAccessClient/Client.h')
0 files changed, 0 insertions, 0 deletions