Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#12 "CDDB" lookup of 830b6f0a causes segfault

Ripperx 2.6
closed-fixed
tony mancill
None
5
2004-11-05
2004-01-22
Karl Ulrich
No

When I tried doing a "CDDB" lookup (even though it's
using freedb) of:

Artist: Rahsaan Roland Kirk
Album: I, Eye, Aye
filename in .cddbslave: 830b6f0a

Ripperx crashes with a Segmentation fault.

This happens if I manually press the "CDDB" button or
have it do the lookup on startup.

I've ripped at least 30 other CDs with Ripperx and not
had any problems. Other applications that do freedb
lookups on my machine don't seem to have any problems
with this disc.

This happens regardless if the 830b6f0a file is present
in .cddbslave or not. I guess that file is actually
created by another application on my box.

Restarting my machine doesn't fix the problem. This is
an RH9 box.

I've attatched the 830b6f0a file, if that helps.

Discussion

  • Karl Ulrich
    Karl Ulrich
    2004-01-22

    .cddbslave file entry for CD causing segfault

     
    Attachments
  • tony mancill
    tony mancill
    2004-10-11

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

    Logged In: YES
    user_id=212006

    It looks like ripperX is puking when it sees the extended
    name for track 7 (EXTT7=a. Satin Doll\nb. Improvisation). I
    imagine this is because ripperX is trying to free() that
    EXTT7 line, but only allocated an array long enough to hold
    the (first 9) primary track names.

    Thanks!

     
  • tony mancill
    tony mancill
    2004-10-12

    Logged In: YES
    user_id=212006

    As it turns out, this is much simpler than I initially
    through. According to the CDDB documentation
    (http://www.freedb.org/modules.php?name=Sections&sop=viewarticle&artid=29),
    the 830b6f0a is incorrectly formatted. Specifically, there
    should be a space on either side of the '/' character
    separating the artist and disc title.

    Still, ripperX shouldn't segfault. I'm working on a patch
    for it now.

     
  • Karl Scriba
    Karl Scriba
    2004-10-18

    Logged In: YES
    user_id=581926

    Using ripperX 2.6.2 the problem description also applies to
    misc/d511e40f and rock/b611af0e.

     
  • tony mancill
    tony mancill
    2004-10-25

    Logged In: YES
    user_id=212006

    the (2) Dio CDs listed in scriba's followup have the same
    problem that the Rahsaan Roland Kirk CDDB entry has - they
    are missing the sequence " / " in the DTITLE line. The
    2.6.3 patch release has a fix that will hopefully resolve
    this issue.

     
  • tony mancill
    tony mancill
    2004-11-05

    • status: open --> closed-fixed
     
  • tony mancill
    tony mancill
    2004-11-05

    Logged In: YES
    user_id=212006

    things should be better in 2.6.3