[SRU] virtualbox crash in raw mode

Bug #1456558 reported by Gianfranco Costamagna
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
virtualbox (Debian)
Fix Released
Unknown
virtualbox (Ubuntu)
Fix Released
Medium
Unassigned
Trusty
Fix Released
Medium
Unassigned
Utopic
Won't Fix
Medium
Unassigned

Bug Description

[Impact]
Crash in raw mode: cfr https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=785689

[Test case]
Run an image in raw mode for around 10 minutes

[Regression potential]
None, from upstream and later releases (vivid included)

Tags: patch
Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :
Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :
Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

Please wait for the security upload to be performed (VENOM)

Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

ping, the security upload has been performed :)

Revision history for this message
Martin Pitt (pitti) wrote :

This is fixed in wily. LocutusOfBorg, please add the affected series, it's a lot of work for a sponsor to check which release is still affected.

Changed in virtualbox (Ubuntu):
status: New → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

For trusty/utopic, please add a bug description and the SRU information. In particular, a justification -- how bad is that? Is it worth fixing this in utopic which will be EOL in some 6 weeks? (Trusty is LTS and a different matter, it makes sense to fix non-critical bugs there).

Changed in virtualbox (Ubuntu Utopic):
status: New → Incomplete
Changed in virtualbox (Ubuntu Trusty):
status: New → Incomplete
summary: - virtualbox crash in raw mode
+ [SRU] virtualbox crash in raw mode
description: updated
Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

Upstream asked me to fix it, and I agree a crash avoidable with one line patch is worth an upload.

you are the Sponsor, so I would like to leave up to you the possible fix for the kernel module build, since we are using linux-lts-vivid stack
http://anonscm.debian.org/cgit/pkg-virtualbox/virtualbox.git/commit/?id=28524c4dabd28a27c947eb0bde322272b4729462

do you think you can sponsor also this patch?
(fine for me leaving out utopic)

Mathew Hodson (mhodson)
description: updated
Changed in virtualbox (Ubuntu Trusty):
status: Incomplete → Confirmed
Changed in virtualbox (Ubuntu Utopic):
status: Incomplete → Confirmed
Changed in virtualbox (Debian):
status: Unknown → Fix Released
Revision history for this message
Daniel Holbach (dholbach) wrote :

Utopic is EOL.

Changed in virtualbox (Ubuntu Trusty):
status: Confirmed → Won't Fix
status: Won't Fix → Confirmed
Changed in virtualbox (Ubuntu Utopic):
status: Confirmed → Won't Fix
Mathew Hodson (mhodson)
tags: added: patch
Changed in virtualbox (Ubuntu):
importance: Undecided → High
Changed in virtualbox (Ubuntu Trusty):
importance: Undecided → High
Changed in virtualbox (Ubuntu Utopic):
importance: Undecided → High
Mathew Hodson (mhodson)
Changed in virtualbox (Ubuntu):
importance: High → Medium
Changed in virtualbox (Ubuntu Trusty):
importance: High → Medium
Changed in virtualbox (Ubuntu Utopic):
importance: High → Medium
Revision history for this message
Sebastien Bacher (seb128) wrote :

sponsored the trusty debdiff, unsubscribing sponsors then since it was the only remaining one

Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello LocutusOfBorg, or anyone else affected,

Accepted virtualbox into trusty-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/virtualbox/4.3.10-dfsg-1ubuntu6 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in virtualbox (Ubuntu Trusty):
status: Confirmed → Fix Committed
tags: added: verification-needed
Revision history for this message
Sebastien Bacher (seb128) wrote :

bah, the trusty change fails to build

"/«PKGBUILDDIR»/src/VBox/Devices/PC/DevACPI.cpp: In function 'void acpiR3Relocate(PPDMDEVINS, RTGCINTPTR)':
/«PKGBUILDDIR»/src/VBox/Devices/PC/DevACPI.cpp:2847:12: error: 'ACPIState' has no member named 'pDevInsRC'
     pThis->pDevInsRC = PDMDEVINS_2_RCPTR(pDevIns);"

Revision history for this message
Gianfranco Costamagna (costamagnagianfranco) wrote :

this patch doesn't work :( I remember I tested it a while ago, but something might have been changed in the last few months...

I'm going to SRU to a new virtualbox release, that will fix also this one

tags: added: verification-failed
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package virtualbox - 4.3.34-dfsg-1+deb8u1ubuntu1.14.04.1

---------------
virtualbox (4.3.34-dfsg-1+deb8u1ubuntu1.14.04.1) trusty-security; urgency=medium

  * New upstream bugfix release. (LP: #1517161)
    - drop patches and rebase with debian jessie-security upload
    - all CVEs are fixed in the upstream import, drop all the CVE
      patches.

 -- Gianfranco Costamagna <email address hidden> Tue, 17 Nov 2015 14:44:04 +0100

Changed in virtualbox (Ubuntu Trusty):
status: Fix Committed → Fix Released
Mathew Hodson (mhodson)
tags: removed: verification-failed
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.