apport-collect SHOULD prune out /home/%USER/ from JournalErrors

Bug #1739532 reported by fermulator
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
High
Unassigned

Bug Description

During https://bugs.launchpad.net/ubuntu/+source/gnome-shell/+bug/1739525, I ran apport-collect.

I was happy to see that my hostname from the system logs was pruned to "hostname". Great!

However, there are some logs that complain about /home/FOO user ... we SHOULD NOT leak a user's $HOME directory contents (a potential list of local user accounts) into these reports. This MAY be considered as sensitive information.

The JournalErrors.txt should prune it.

Example of CULPRITS:
{{{
Dec 20 21:39:20 hostname com.ubuntu.OneConf[3069]: WARNING:oneconf.hosts:Error in loading other_hosts file: [Errno 2] No such file or directory: '/home/FOO/.cache/oneconf/1dfe6d2e52223c637c7bddd900000002/other_hosts'
Dec 20 21:39:33 hostname com.ubuntu.OneConf[18688]: WARNING:oneconf.hosts:Error in loading other_hosts file: [Errno 2] No such file or directory: '/home/BAR/.cache/oneconf/1dfe6d2e52223c637c7bddd900000002/other_hosts'
}}}

The suggestion here, is simply to also prune out usernames from ANY "/home/%USER" or "~%USER" type regexes.

fermulator (fermulator)
description: updated
Revision history for this message
Brian Murray (brian-murray) wrote :

To be clear I understand your issue here to be that only your username (the logged in user) was replaced in the file JournalErrors.txt and that you'd like to see every username replaced in JournalErrors.txt. Is that a correct statement?

Changed in apport (Ubuntu):
status: New → Incomplete
importance: Undecided → High
tags: added: beaver
tags: added: vionic
tags: added: bionic
removed: vionic
Revision history for this message
Brian Murray (brian-murray) wrote :

I recently modified apport so JournalErrors.txt actually consists of errors and not also warnings and I believe that resolves this specific issue although we could do a better job of searching for and replacing user home directory names.

apport (2.20.8-0ubuntu9) bionic; urgency=medium

  * data/general-hooks/generic.py: change JournalErrors to contain errors not
    warnings. (LP: #1738581)

Changed in apport (Ubuntu):
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.