Menu

#11 CDDB File Flaw in 2.6

Ripperx 2.6
closed-accepted
None
5
2004-10-11
2004-01-03
No

Hi, I like the program and have been using it since 2.4 under Linux.
Just noticed 2.6 today, grabbed source and have noticed that it
did not pay any attention to the files I'd already downloaded into
my ~/.cddb -> ~/.cddbslave directory. After adding a few print stmts
I found a missing line in the cddb.c file. You need to add a
line after line 288:
strcat( file_check, "/" );
to separate the .cddbslave name from the subdirectory name.

A change that I always make in your source is to display the
CDDB ID# of the file that the program is looking for. Since it
occasionally couldn't find the CDDB file, yet the CD player could,
it is handy to be able to determine exactly what file it is trying to use.

Thanks for your work!

Steve Bresson

Discussion

  • tony mancill

    tony mancill - 2004-10-11
    • assigned_to: nobody --> tmancill
     
  • tony mancill

    tony mancill - 2004-10-11

    Logged In: YES
    user_id=212006

    This patch is included in the 2.6.2 release. Thank you for
    uncovering this.

     
  • tony mancill

    tony mancill - 2004-10-11
    • status: open --> closed-accepted
     

Log in to post a comment.