Could you please attach or make the file available?

I just noticed that its not the mp3 that is the problem, its the playlist. 

If i drag the folder to Totem, Totem opens the playlist (m3u). If i then click on a entry with 'weird' encoding Totem crashes. Attaching the m3u file and putting the mp3s up on my webserver. (© issues) :)

Mp3s:
http://internethemsidan.se/Beach%20Party%20CD2/

Created attachment 92743 [details]
m3u file

You uploaded Beach Party CD2, where the crash occurred on CD1. I don't see anything wrong with the playlist either, it's getting parsed properly, though we don't seem to escape the filenames properly.

I'm pretty sure that if there is a problem, it's due to the filename, so please attach the file that made it crash in a tar file, so I can get the filename untouched.

Here you go:
http://www.internethemsidan.se/mp3-file.tar

Times out, I can't download.

Vacation in Spain for a week, up and running again.

I tried it, and I don't see any problems at all...

In gdb, once you have the backtrace, could you make it print:
subtitle
suffix
i
subtitle_ext[i]

I could if i knew how to do it :)

What shall i type into the gdb console?

frame 0
p subtitle
p suffix
etc.

(p being short for "print" :)

Did what you asked, with some combos because some of your instructions didn't work. Hope it helps.

(gdb) frame 0

+ Trace 154035

$1 = 0x0
(gdb) p suffix
$2 = <value optimized out>
(gdb) p i
No symbol "i" in current context.
(gdb) p subtitle_ext[i]
No symbol "i" in current context.
(gdb) p 1
$3 = 1
(gdb) p 3
$4 = 3
(gdb) p subtitle_ext
$5 = {"asc", "txt", "sub", "srt", "smi", "ssa"}

That's the code it's crashing in:
        subtitle = g_strdup_printf ("%.*s.???", suffix, uri);

        /* Search for any files with one of our known subtitle extensions */
        for (i = 0; i < G_N_ELEMENTS(subtitle_ext) ; i++) {
                memcpy (subtitle + suffix + 1, subtitle_ext[i], 3);

This means g_strdup_printf() fails (?!). I've added a warning in there. Could you please test against the current SVN and let me know if you still see the crash or just a warning?

(In reply to comment #12)
> That's the code it's crashing in:
>         subtitle = g_strdup_printf ("%.*s.???", suffix, uri);
> 
>         /* Search for any files with one of our known subtitle extensions */
>         for (i = 0; i < G_N_ELEMENTS(subtitle_ext) ; i++) {
>                 memcpy (subtitle + suffix + 1, subtitle_ext[i], 3);
> 
> This means g_strdup_printf() fails (?!). I've added a warning in there. Could
> you please test against the current SVN and let me know if you still see the
> crash or just a warning?
> 

Totem does not crash anymore BUT, it won't play the file. It says something like "An error has occurred
Could not find the location"


Gdb-output:
** (totem:32373): CRITICAL **: totem_uri_get_subtitle_uri: assertion `subtitle != NULL' failed
0:01:31.727034000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:310:bvw_error_msg: message = Resursen kunde inte hittas.
0:01:31.727073000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:312:bvw_error_msg: domain  = 2016 (gst-resource-error-quark)
0:01:31.727088000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:313:bvw_error_msg: code    = 3
0:01:31.727099000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:314:bvw_error_msg: debug   = gstfilesrc.c(971): gst_file_src_start (): /play/source:
No such file "/home/filip/Music/Beach Party (2007)/Beach Party CD1/04 - Tomas Ledin - En Dag P� Stranden.mp3"
0:01:31.727120000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:315:bvw_error_msg: source  = <source>
0:01:31.727142000 32373 0x809a470 ERROR                totem bacon-video-widget-gst-0.10.c:316:bvw_error_msg: uri     = file:///home/filip/Music/Beach%20Party%20(2007)/Beach%20Party%20CD1/04 - Tomas Ledin - En Dag P� Stranden.mp3
** Message: Error: Resursen kunde inte hittas.
gstfilesrc.c(971): gst_file_src_start (): /play/source:
No such file "/home/filip/Music/Beach Party (2007)/Beach Party CD1/04 - Tomas Ledin - En Dag P\xe5 Stranden.mp3"

I don't know why it wouldn't copy the URI. There's also a bug in the playlist parser in that the m3u parser doesn't escape the lines it gets from the file.

Can you put the m3u file and the second mp3 up on a more reliable host, so that I can carry on testing?

(In reply to comment #14)
> I don't know why it wouldn't copy the URI. There's also a bug in the playlist
> parser in that the m3u parser doesn't escape the lines it gets from the file.
> 
> Can you put the m3u file and the second mp3 up on a more reliable host, so that
> I can carry on testing?
>
Here you go...

M3u file + the second mp3:
http://virkenstick.se/bugzilla/m3u+02.tar

M3u file + the first four mp3:
http://virkenstick.se/bugzilla/m3u+4mp3.tar

Fixed the "subtitle is NULL" problem.

2007-08-15  Bastien Nocera  <hadess@hadess.net>

        * src/totem-uri.c: (totem_uri_get_subtitle_uri): Remove
        the "%.*s" construct, it's completely broken with wide-characters
        and non-UTF8 ones

Fixed now.

However, your tarball contained the file with the UTF-8 encoding (so didn't match the ISO-8859-1 name given in the m3u).

The URI as in the M3U file:
file:///home/hadess/Music/non-utf8-test/04%20-%20Tomas%20Ledin%20-%20En%20Dag%20P%E5%20Stranden.mp3
The real URI that can be played from (DND from Nautilus):
file:///home/hadess/Music/non-utf8-test/04%20-%20Tomas%20Ledin%20-%20En%20Dag%20P%C3%A5%20Stranden.mp3

2007-08-15  Bastien Nocera  <hadess@hadess.net>

        * src/plparse/totem-pl-parser-lines.c:
        (totem_pl_parser_append_path), (totem_pl_parser_add_m3u):
        Escape the relative paths found in m3u files, as they're filenames
        not URIs (Closes: #461951)