Patch for FixupTOC

Hi
I think I've found a bug (or maybe incompatibility with my cd-drive.) I
tried ripping a cd of mine with a pre-track, and cdparanoia just went
nuts. It seemed cdparanoia thought the last track had a negative number
of sectors, which made it impossible to get anything from the cd except
a part of the pre-track. I tracked down the bug to the multi-session
fix in FixupTOC, and I've attach my patch. I don't say this is the Right
Way to do it (haven't tested against a multi-session cd), but I don't
think it will cause any incompabilities.

Otherwice it's a GREAT TOOL. Thanx

--

-- 
Erik Månsson

 Zapp: Now that's a wave of destruction that's easy on the eyes. 

diff -urN cdparanoia-III-alpha9.8/interface/common_interface.c cdparanoia-III-alpha9.8.new/interface/common_interface.c
--- cdparanoia-III-alpha9.8/interface/common_interface.c	2000-04-20 00:40:42.000000000 +0200
+++ cdparanoia-III-alpha9.8.new/interface/common_interface.c	2002-11-06 23:06:03.000000000 +0100
 <at>  <at>  -223,7 +223,8  <at>  <at> 
       /* adjust end of last audio track to be in the first session */
       for (j = tracks-1; j >= 0; j--) {
 	if (j > 0 && !IS_AUDIO(d,j) && IS_AUDIO(d,j-1)) {
-	  if (d->disc_toc[j].dwStartSector > ms_str.addr.lba - 11400) 
+	  if ((d->disc_toc[j].dwStartSector > ms_str.addr.lba - 11400) &&
+            (ms_str.addr.lba - 11400 > d->disc_toc[j-1].dwStartSector))
 	    d->disc_toc[j].dwStartSector = ms_str.addr.lba - 11400;
 	  break;

Issue with Peter Gabriel's "Up"

This isn't really a bug report, but I figure you might find this
interesting.

I tried to rip Peter Gabriel's Up using cdparanoia (via grip), and it
didn't work -- it just skipped right through the tracks in order without
reading any info, as if the tracks were 0 seconds long. I switched to
cdda2wav and it ripped fine. To date, I've probably ripped 100+ discs
with cdparanoia without a problem.

In addition, when I initially put the cd into a desktop cd-rom, it
couldn't read it at all, and somehow fucked up my cd-rom so that it
couldn't read any audio cds until I went through about 5 power cycles. 

I figure it might be copy-protection, or a weird by-product of the media
enhancement on the cd, but I don't really know.

Version info:
Omnibook 6000
Vanilla Redhat 8.0
Kernel 2.4.18-18.8.0
cdparanoia 9.8
CD-ROM drive: "Matshita DVD-ROM SR-8175" running under scsi-emulation

--james

ps. Keep up the good work.
--

-- 
  James D Thomas
  jthomas <at> kodos.fastmail.fm
--- >8 ----

yet another cddb patch

Hello,

I'm an over-satisfied user of cdparanoia, and I've made a patch to
fetch track titles from freedb databases. I know this will not be
included in the release of cdparanoia, since it is not its goal, but
if someone need it, it applies against cdparanoia-III-alpha-9.8 
(couldn't find cdparanoia-IV to test).

Cheers, /T

Final Scratch

I have been using final scratch for a month. Everything was fine until 
one day, I could no longer import songs (rip from cd). I noticed your 
website when I was shutting down the program. Is your cd paranoia the 
ripping program for final scratch? If so, how can I fix the problem? 
Final Scratch is useless if I can't load songs...
--DJ Low  ***King of the Megamixxx***           Mud Funk 4eva Bia 
Bia!!!
Flava House Music 601 939-5456
--- >8 ----
List archives:  http://www.xiph.org/archives/
Paranoia homepage: http://www.xiph.org/paranoia/
To unsubscribe from this list, send mail to 'paranoia-dev-request <at> xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.

Re: Patch for FixupTOC

On Wed, Nov 06, 2002 at 11:21:21PM +0100, goodbyte <at> phreaker.net wrote:
> Hi
> I think I've found a bug (or maybe incompatibility with my cd-drive.) I
> tried ripping a cd of mine with a pre-track, and cdparanoia just went
> nuts. It seemed cdparanoia thought the last track had a negative number
> of sectors, which made it impossible to get anything from the cd except
> a part of the pre-track. I tracked down the bug to the multi-session
> fix in FixupTOC, and I've attach my patch. I don't say this is the Right
> Way to do it (haven't tested against a multi-session cd), but I don't
> think it will cause any incompabilities.
> 
> Otherwice it's a GREAT TOOL. Thanx

Patch accepted and committed.  thank you for flying cdparanoia airlines!

Monty
--- >8 ----
List archives:  http://www.xiph.org/archives/
Paranoia homepage: http://www.xiph.org/paranoia/
To unsubscribe from this list, send mail to 'paranoia-dev-request <at> xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.