adplug+libbinio+memory mapped = crash on some files

Bug #1839038 reported by Stian Skjelstad
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libbinio (Debian)
Fix Released
Unknown
libbinio (Ubuntu)
Fix Released
Undecided
Unassigned
ocp (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The build already includes this patch:

https://github.com/adplug/libbinio/commit/76b5346a6196c76a1e9ea1ae0593d695da1dbd37

but it will also need this to remove all crashes (".D00" files, that are memory mapped for instance, when loaded via libadplug)

https://github.com/adplug/libbinio/pull/8/commits/41fddc3a8123ec5455dcc8979f34a8b705eee5e6

(The crashes that happens inside Adplug, is due to memory allocated buffer that only receives the first byte set from a read-call, and further parsing assumes that the full data has been fetched)

Revision history for this message
Stian Skjelstad (mywave) wrote :

https://github.com/adplug/libbinio has now actually just tagged version 1.5

libbinio1v5 can now be upgraded to build on this release instead of 1.4 + cherry-picked patches, that also includes this fix

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in libbinio (Ubuntu):
status: New → Confirmed
Revision history for this message
Stian Skjelstad (mywave) wrote :

OpenCubicPlayer next version depends on this getting fixed

Revision history for this message
Lenin (gagarin) wrote :

Indeed, being the maintainer of ocp, I'm interested getting it updated, including adplug support. Builds also without, loses a lot of supported files though (adplug)

Changed in ocp (Ubuntu):
status: New → Confirmed
Revision history for this message
Lenin (gagarin) wrote :

a fixed source deb package is available http://phd-sid.ethz.ch/debian/libbinio/ since oct-2019

Revision history for this message
Lenin (gagarin) wrote :

amoog would you mind fixing it please? or allow me join you as a team member to fix it?

Revision history for this message
Lenin (gagarin) wrote :

summary: ocp+libbinio 1.5 with adlib player functionality:

http://phd-sid.ethz.ch/debian/libbinio/
http://phd-sid.ethz.ch/debian/ocp/2020/

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

Fix uploaded in Ubuntu focal.
Debian is in deferred/10

Changed in libbinio (Ubuntu):
status: Confirmed → Fix Released
Changed in ocp (Ubuntu):
status: Confirmed → Fix Released
Changed in libbinio (Debian):
importance: Undecided → Unknown
status: New → Unknown
Changed in libbinio (Debian):
status: Unknown → Fix Committed
Changed in libbinio (Debian):
status: Fix Committed → 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.