autopkgtests fail after security fixes
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Apport |
Fix Released
|
Undecided
|
Unassigned | ||
apport (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Xenial |
Fix Released
|
High
|
Unassigned | ||
Bionic |
Fix Released
|
High
|
Unassigned | ||
Disco |
Won't Fix
|
High
|
Unassigned | ||
Eoan |
Fix Released
|
High
|
Unassigned |
Bug Description
The following autopkgtests fail after the recent security fixes:
log:FAIL: test_get_
log:FAIL: test_core_
log:FAIL: test_core_
log:FAIL: test_crash_
log:FAIL: test_crash_
log:FAIL: test_crash_
log:FAIL: test_lock_symlink (__main__.T)
test_get_
test_crash_setuid_* are caused by the dropping of privileges when accessing the crashing process's /proc. They seem to be testing behaviour now explicitly forbidden by the fix to be honest!
test_lock_symlink fails because the lock file is now always in /var/lock/apport/ and not in $APPORT_REPORT_DIR. I guess we could update the test, but is it really worth it after the fix?
Related branches
- Apport upstream developers: Pending requested
-
Diff: 47 lines (+6/-5)2 files modifieddata/apport (+3/-4)
test/test_report.py (+3/-1)
- Brian Murray: Approve
-
Diff: 220 lines (+49/-47)6 files modifiedapport/report.py (+19/-10)
data/apport (+13/-6)
debian/changelog (+12/-0)
etc/init.d/apport (+2/-2)
test/test_report.py (+3/-1)
test/test_signal_crashes.py (+0/-28)
tags: | added: update-excuse update-excuse-eoan |
Changed in apport (Ubuntu): | |
status: | New → In Progress |
Changed in apport (Ubuntu): | |
importance: | Undecided → High |
Changed in apport (Ubuntu Xenial): | |
importance: | Undecided → High |
Changed in apport (Ubuntu Bionic): | |
importance: | Undecided → High |
Changed in apport (Ubuntu Disco): | |
importance: | Undecided → High |
Changed in apport (Ubuntu Eoan): | |
importance: | Undecided → High |
tags: | added: id-5dbd08153201707b96d7796f |
Changed in apport (Ubuntu Disco): | |
status: | New → Won't Fix |
Changed in apport: | |
status: | New → Fix Released |
test_crash_setuid_* should only test for that else clause now.