Commit f1dd15a8 authored by Allen Webb's avatar Allen Webb Committed by Commit Bot

Remove "system_files" from CommandLineLogSource.

The system logs command to retrieve a list of system files is broken by
a seccomp violation and is leading to crashes. Since it has been broken
for a while and there haven't been complaints about the missing data,
this change removes the broken command.

Bug: 1021425
Change-Id: Ia8c5ab094bd1e97bb2996d31c458a00657c8b9b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2298707
Auto-Submit: Allen Webb <allenwebb@google.com>
Reviewed-by: default avatarThiemo Nagel <tnagel@chromium.org>
Reviewed-by: default avatarAhmed Fakhry <afakhry@chromium.org>
Commit-Queue: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#788792}
parent 4766ddd6
......@@ -61,19 +61,6 @@ void ExecuteCommandLines(system_logs::SystemLogsResponse* response) {
command = base::CommandLine((base::FilePath("/usr/bin/printenv")));
commands.emplace_back("env", command);
// Get a list of file sizes for the whole system (excluding the names of the
// files in the Downloads directory for privay reasons).
if (base::SysInfo::IsRunningOnChromeOS()) {
// The following command would hang if run in Linux Chrome OS build on a
// Linux Workstation.
command = base::CommandLine(base::FilePath("/bin/sh"));
command.AppendArg("-c");
command.AppendArg(
"/usr/bin/du -h --max-depth=5 /home/ /mnt/stateful_partition/ | "
"grep -v -e Downloads -e IndexedDB -e databases");
commands.emplace_back("system_files", command);
}
// Get disk space usage information
command = base::CommandLine(base::FilePath("/bin/df"));
commands.emplace_back("disk_usage", command);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment