bugCompact Disc Input and Control Library - Bugs: bug #49831, libparanoia reads incorrect track...

 
 

bug #49831: libparanoia reads incorrect track with full paranoia, correct track with -Z and -Y

Submitted by:  Samantha Baldwin <fuhsaz>
Submitted on:  Mon 12 Dec 2016 11:35:50 PM UTC  
 
Category: NoneSeverity: 5 - Average
Item Group: libparanoiaStatus: Fixed
Privacy: PublicAssigned to: Robert Kausch <enzo1982>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 13 Dec 2016 12:20:29 AM UTC, comment #1:

Thank you for reporting this! I'll have a look at it.

Robert Kausch <enzo1982>
Project MemberIn charge of this item.
Mon 12 Dec 2016 11:35:50 PM UTC, original submission:

Greetings. This is a duplicate of a bug report I made on the paranoia-dev[0] mailing list. However, cdparanoia is essentially abandoned at this point, so you guys get the bug.

What has changed in the interim, though, is that someone who wishes to remain anonymous (NOT ME) has contributed a fix[1].

The crux of the bug, according to the anonymous developer:

"When analyzing stage 2 of the ripping process and looking at what happens differently with the previous patch applied and not applied, I discovered that cdparanoia detected a rift when it was not applied but not when it was applied. This led me to the try_sort_sync function. Through some trial and error, I found that in all stage 2 calls of this function the variable zeropos is 10 when a match is found, except for where the error occurs, there it is -128. This will cause the try_sort_sync function to go into a mode where it tries to match other positions. So the problem is that there is some identical data that is matched that shouldn't be. The offset for this data is then wrong, which will result in jitter being reported (this is why there is a ! in the output). This jitter is seen as a rift, which cdparanoia tries to fix. Since the rift isn't actually there, the fixing makes incorrect merges."

They have contributed two potential fixes:

"Pretty sure I fixed the cdparanoia bug. It actually wasn't in a silence block. There were 2 identical 109 samples of data in the track right near each other. cdparanoia was matching the second one with the first one and saying it was jitter.

This fix doesn't lose out on any error correcting capablities. In the case of multiple matching runs it picks the one with the smallest (absolute value) offset (diff 1) or the one with the longest match (diff 2). Since multiple matching runs should be extremely rare, the impact of this change on other rips should be almost nonexistent. Both diffs fix the Julie Roberts track 5 bug."

[0] http://lists.xiph.org/pipermail/paranoia-dev/2016-August/000279.html
[1] http://lists.xiph.org/pipermail/paranoia-dev/2016-December/000280.html

Samantha Baldwin <fuhsaz>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #39207:  cdparanoia-longest-match.patch added by fuhsaz (2KiB - application/octet-stream)
file #39208:  cdparanoia-smallest-offset.patch added by fuhsaz (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rocky (Updated the item)
  • -unavailable- added by enzo1982 (Posted a comment)
  • -unavailable- added by fuhsaz (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 25 Mar 2017 09:46:33 AM UTCrockyStatusIn Progress=>Fixed
      Open/ClosedOpen=>Closed
    Tue 13 Dec 2016 12:20:29 AM UTCenzo1982Item GroupNone=>libparanoia
      StatusNone=>In Progress
      Assigned toNone=>enzo1982
    Mon 12 Dec 2016 11:35:50 PM UTCfuhsazAttached File-=>Added cdparanoia-longest-match.patch, #39207
      Attached File-=>Added cdparanoia-smallest-offset.patch, #39208

    Back to the top


    Powered by Savane 3.1-cleanup1