SIGSEGV when file2str reads zero bytes
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
procps (Ubuntu) |
Fix Released
|
Medium
|
Dave Chiluk | ||
Precise |
Fix Released
|
Undecided
|
Dave Chiluk | ||
Quantal |
Fix Released
|
Undecided
|
Dave Chiluk | ||
Raring |
Fix Released
|
Undecided
|
Dave Chiluk | ||
Saucy |
Fix Released
|
Undecided
|
Dave Chiluk |
Bug Description
[Impact]
* various procps utilities crashing with a SIGSEGV if the file2str function in
proc/readproc.c when it reads zero bytes
* This is a regression introduced with (LP: #1150413)
* The upload checks for zero length reads and now returns -1 instead of 0, like it used
to.
[Test Case]
* This does not happen often enough to create a real testcase, but some users have
reported that repeatedly running top has occassionally exhibited this problem.
[Regression Potential]
* Minimal, as this reverts file2str to original behavior in the case of zero length
reads.
[Other Info]
* This patch is backported from upstream
_______
The changes made in the following commit, which backported a number of changes to the procps package in precise, result in various procps utilities crashing with a SIGSEGV if the file2str function in proc/readproc.c reads zero bytes.
http://
This corresponds to the following procps package version: 1:3.2.8-11ubuntu6.1
Prior to this changeset, file2str would return -1 if the read failed; now it does not, which is not expected by other parts of the procps codebase, hence the crash.
Upstream have fixed this issue in the following commit:
https:/
Changed in procps (Ubuntu): | |
status: | Triaged → In Progress |
Changed in procps (Ubuntu Precise): | |
status: | New → In Progress |
Changed in procps (Ubuntu Quantal): | |
status: | New → In Progress |
Changed in procps (Ubuntu Raring): | |
status: | New → In Progress |
Changed in procps (Ubuntu Precise): | |
assignee: | nobody → Dave Chiluk (chiluk) |
Changed in procps (Ubuntu Quantal): | |
assignee: | nobody → Dave Chiluk (chiluk) |
Changed in procps (Ubuntu Raring): | |
assignee: | nobody → Dave Chiluk (chiluk) |
description: | updated |
tags: |
added: verification-done-precise verification-needed removed: verification-done |
Changed in procps (Ubuntu Saucy): | |
assignee: | nobody → Dave Chiluk (chiluk) |
Triaged: Pointer to upstream fix
John-Mark: How often does this trigger, is this something that causes regular pain or a rare issue?