** Description changed:
- Cached pid info is added to the hash table even when it's a re-used pid
- info causing a double free on exit cleanup. Only add pid info to hash
- table if it is newly allocated.
+ [XENIAL SRU]
+
+ [Impact]
+ Cached pid info is added to the hash table even when it's a re-used pid info
causing a double free on exit cleanup. Only add pid info to hash table if it
is newly allocated.
+
+ [Test Case]
+ Run cpustat for a long duration as follows:
+
+ cpustat 1 -T -g
+
+ ..and work through a lot of processes IDs until we re-use a PID, e.g.:
+
+ stress-ng --fork 0 &
+
+ and stop cpustat with SIGINT, one hits a double free as it exits
+
+ [Regression Potential]
+
+ Minimal, the fix stops an item being added to a cache'd hash list more
+ than once, and that is a minimal impact on the main cpustat
+ functionality.
+
+ [Fix]
+ Upstream commit
http://kernel.ubuntu.com/git/cking/cpustat.git/commit/?id=dc7d2fe45560c414645bc286a7cd63f13b69dc6c
** Also affects: cpustat (Ubuntu Xenial)
Importance: Undecided
Status: New
** Changed in: cpustat (Ubuntu Xenial)
Status: New => In Progress
** Changed in: cpustat (Ubuntu Xenial)
Importance: Undecided => High
** Changed in: cpustat (Ubuntu Xenial)
Assignee: (unassigned) => Colin Ian King (colin-king)
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1631050
Title:
cache'd pid recycling adds cached info to hash again causing double
free on exit cleanup
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/cpustat/+bug/1631050/+subscriptions
--
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
