tag 407699 + patch
thanks

According to GDB, this is the back trace:

        #0  mp3playlist::count (this=0x8060000) at playlist.cc:370
        No locals.
        #1  0x0804b6f6 in prepare_new_list () at main.cc:222
                c = <value optimized out>
        #2  0x0804bf18 in play_list () at main.cc:446
                playing = true
                s = 1
        #3  0x0804e005 in main (argc=Cannot access memory at address 0xfffffffa
        ) at main.cc:916
                addsuccess = <value optimized out>
                ch = <value optimized out>

However this is incorrect. By stepping through with the debugger I
arrived at:

        0 playlist.cc:370  mp3playlist::count
        1 interface.cc:300 playmp3listWindow::draw_list
        2 main.cc:213      prepare_new_list
        3 main.cc:446      play_list
        4 main.cc:916      main

I rebuilt the package without optimisations and found that this fixes
the bug. Please review the attached patch.

-- 
Sam Morris
http://robots.org.uk/

PGP key id 1024D/5EA01078
3412 EA18 1277 354B 991B  C869 B219 7FDB 5EA0 1078
diff -u playmp3list-0.95/debian/changelog playmp3list-0.95/debian/changelog
--- playmp3list-0.95/debian/changelog
+++ playmp3list-0.95/debian/changelog
@@ -1,3 +1,10 @@
+playmp3list (0.95-4.2) UNRELEASED; urgency=high
+
+  * Non-maintainer upload.
+  * Disable optimisations (closes: #407699)
+
+ -- Sam Morris <[EMAIL PROTECTED]>  Sat, 10 Mar 2007 12:54:51 +0000
+
 playmp3list (0.95-4.1) unstable; urgency=low
 
   * NMU.
diff -u playmp3list-0.95/debian/rules playmp3list-0.95/debian/rules
--- playmp3list-0.95/debian/rules
+++ playmp3list-0.95/debian/rules
@@ -8,6 +8,10 @@
 # This is the debhelper compatability version to use.
 export DH_COMPAT=2
 
+# Disabling optimisations prevents a crash when changing directories
+# http://bugs.debian.org/407699
+export CXXFLAGS = -g -O0
+
 configure: configure-stamp
 configure-stamp:
 	dh_testdir

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to