0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Kernel: Make sure JSON blobs in core dumps are correctly terminated

Browse source 
I regressed this in 648480f715.
We have to make sure JsonObjectSerializer::finish() is called before
writing out the blob. This is done automatically when the serializer
is destroyed, so just wrap them in scopes.
This commit is contained in:
Andreas Kling 2021-06-29 22:27:11 +02:00
parent e2187f2956
commit bdc905ab93
Notes: sideshowbarker 2024-07-18 11:15:49 +09:00
1 changed files with 20 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
Kernel/CoreDump.cpp
View file

@ -207,21 +207,23 @@ ByteBuffer CoreDump::create_notes_process_data() const
process_data.append((void*)&info, sizeof(info));
StringBuilder builder;
JsonObjectSerializer process_obj { builder };
process_obj.add("pid"sv, m_process->pid().value());
process_obj.add("termination_signal"sv, m_process->termination_signal());
process_obj.add("executable_path"sv, m_process->executable() ? m_process->executable()->absolute_path() : String::empty());
{
auto arguments_array = process_obj.add_array("arguments"sv);
for (auto& argument : m_process->arguments())
arguments_array.add(argument);
}
JsonObjectSerializer process_obj { builder };
process_obj.add("pid"sv, m_process->pid().value());
process_obj.add("termination_signal"sv, m_process->termination_signal());
process_obj.add("executable_path"sv, m_process->executable() ? m_process->executable()->absolute_path() : String::empty());
{
auto environment_array = process_obj.add_array("environment"sv);
for (auto& variable : m_process->environment())
environment_array.add(variable);
{
auto arguments_array = process_obj.add_array("arguments"sv);
for (auto& argument : m_process->arguments())
arguments_array.add(argument);
}
{
auto environment_array = process_obj.add_array("environment"sv);
for (auto& variable : m_process->environment())
environment_array.add(variable);
}
}
builder.append(0);
@ -287,9 +289,11 @@ ByteBuffer CoreDump::create_notes_metadata_data() const
metadata_data.append((void*)&metadata, sizeof(metadata));
StringBuilder builder;
JsonObjectSerializer metadata_obj { builder };
for (auto& it : m_process->coredump_metadata())
metadata_obj.add(it.key, it.value);
{
JsonObjectSerializer metadata_obj { builder };
for (auto& it : m_process->coredump_metadata())
metadata_obj.add(it.key, it.value);
}
builder.append(0);
metadata_data.append(builder.string_view().characters_without_null_termination(), builder.length());