system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-10-17 13:22:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

CrashReporter: Don't keep the coredump file open longer than necessary

Browse source 
Let's be nice and close the file after we've extracted what we need.
This commit is contained in:
Andreas Kling 2020-12-30 02:56:31 +01:00
parent 8df1f6951e
commit 10b5b9ee66
1 changed files with 12 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
Applications/CrashReporter/main.cpp
View file

@ -59,10 +59,15 @@ int main(int argc, char** argv)
args_parser.add_positional_argument(coredump_path, "Coredump path", "coredump-path");
args_parser.parse(argc, argv);
auto coredump = CoreDump::Reader::create(coredump_path);
if (!coredump) {
warnln("Could not open coredump '{}'", coredump_path);
return 1;
Optional<CoreDump::Backtrace> backtrace;
{
auto coredump = CoreDump::Reader::create(coredump_path);
if (!coredump) {
warnln("Could not open coredump '{}'", coredump_path);
return 1;
}
backtrace = coredump->backtrace();
}
auto app = GUI::Application::construct(argc, argv);
@ -72,15 +77,13 @@ int main(int argc, char** argv)
return 1;
}
auto backtrace = coredump->backtrace();
String executable_path;
// FIXME: Maybe we should just embed the process's executable path
// in the coredump by itself so we don't have to extract it from the backtrace.
// Such a process section could also include the PID, which currently we'd have
// to parse from the filename.
if (!backtrace.entries().is_empty()) {
executable_path = backtrace.entries().last().object_name;
if (!backtrace.value().entries().is_empty()) {
executable_path = backtrace.value().entries().last().object_name;
} else {
warnln("Could not determine executable path from coredump");
return 1;
@ -143,7 +146,7 @@ int main(int argc, char** argv)
StringBuilder backtrace_builder;
auto first = true;
for (auto& entry : backtrace.entries()) {
for (auto& entry : backtrace.value().entries()) {
if (first)
first = false;
else