Bug 2272548

Summary: Finding offset fails for one drive "file size 0 did not match expected size"
Product: [Fedora] Fedora Reporter: michael <gromic>
Component: libcdio-paranoiaAssignee: Adrian Reber <adrian>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: 40CC: adrian, bugzilla
Target Milestone: ---Keywords: Bugfix, EasyFix, FastFix
Target Release: ---   
Hardware: x86_64   
OS: Linux   
URL: https://github.com/whipper-team/whipper/issues/234
Whiteboard:
Fixed In Version: libcdio-paranoia-10.2+2.0.1-12.fc40 Doc Type: ---
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-04-19 21:26:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description michael 2024-04-01 21:22:21 UTC
Hello maintainers of libcdio-paranoia,

Today I tried to back up an audio compact disc. 
I came across a bug that has been known for a long time (even whipper links the bug report in warning message) but the fix apparently has not yet arrived in Fedora.

I am not able to backup my audio compact disc with bluray drive.

The libcdio-paranoia upstream merged fix: https://github.com/rocky/libcdio-paranoia/pull/38
The whipper bug report: https://github.com/whipper-team/whipper/issues/234


Reproducible: Always

Steps to Reproduce:
1.whipper drive analyze
2.whipper offset find
3.WARNING:whipper.command.offset:cannot rip with offset 667...
Actual Results:  
INFO:whipper.command.offset:trying read offset 6...
INFO:whipper.command.offset:trying read offset 667...
WARNING:whipper.program.cdparanoia:because of a cd-paranoia upstream bug whipper may fail to work correctly when using offset values > 587 (current value: 667) and print warnings like this: 'file size 0 did not match expected size'. For more details please check the following issues: https://github.com/whipper-team/whipper/issues/234 and https://github.com/rocky/libcdio-paranoia/issues/14
WARNING:whipper.program.cdparanoia:file size 0 did not match expected size 66418172
WARNING:whipper.program.cdparanoia:non-integral amount of frames difference
Traceback (most recent call last):        
  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 193, in getTrackQuality
    return min(frames * 2.0 / reads, 1.0)
               ~~~~~~~~~~~~~^~~~~~~
ZeroDivisionError: float division by zero

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib64/python3.12/site-packages/whipper/extern/task/task.py", line 523, in c
    callable_task(*args, **kwargs)
  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 324, in _read
    self._done()
  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 400, in _done
    self.quality = self._parser.getTrackQuality()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 195, in getTrackQuality
    raise RuntimeError("cdparanoia couldn't read any frames "
RuntimeError: cdparanoia couldn't read any frames for the current track
WARNING:whipper.command.offset:unknown task exception for offset 667: (RuntimeError("cdparanoia couldn't read any frames for the current track"), 'exception RuntimeError at /usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py:195: getTrackQuality(): cdparanoia couldn\'t read any frames for the current track\nTraceback (most recent call last):\n  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 193, in getTrackQuality\n    return min(frames * 2.0 / reads, 1.0)\n               ~~~~~~~~~~~~~^~~~~~~\nZeroDivisionError: float division by zero\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File "/usr/lib64/python3.12/site-packages/whipper/extern/task/task.py", line 523, in c\n    callable_task(*args, **kwargs)\n  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 324, in _read\n    self._done()\n  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 400, in _done\n    self.quality = self._parser.getTrackQuality()\n                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File "/usr/lib64/python3.12/site-packages/whipper/program/cdparanoia.py", line 195, in getTrackQuality\n    raise RuntimeError("cdparanoia couldn\'t read any frames "\nRuntimeError: cdparanoia couldn\'t read any frames for the current track\n')
WARNING:whipper.command.offset:cannot rip with offset 667...
INFO:whipper.command.offset:trying read offset 48...

Comment 1 Fedora Update System 2024-04-02 08:58:00 UTC
FEDORA-2024-d751e81722 (libcdio-paranoia-10.2+2.0.1-12.fc40) has been submitted as an update to Fedora 40.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-d751e81722

Comment 2 Fedora Update System 2024-04-03 01:22:19 UTC
FEDORA-2024-d751e81722 has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-d751e81722`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-d751e81722

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 3 Peter Oliver 2024-04-03 23:49:17 UTC
*** Bug 2260787 has been marked as a duplicate of this bug. ***

Comment 4 Fedora Update System 2024-04-19 21:26:53 UTC
FEDORA-2024-d751e81722 (libcdio-paranoia-10.2+2.0.1-12.fc40) has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.