sp_session_starred_create segmentation fault about mopidy-spotify HOT 7 CLOSED

This sounds fantastic. Sounds very much like something not too hard to fix. I'll have a look at this right away :-)

from mopidy-spotify.

I get the same segfault, or at least a very similar one (http://ix.io/k6i), when sending "status\nlsinfo" and sometimes when sending just "lsinfo" to the MPD interface. The lsinfo command has to be without arguments or with just / as the argument. It doesn't happen if I give another argument to lsinfo, I guess because it doesn't load the playlists then.

I tried running mopidy with a pdb breakpoint in get_starred and stepped through the code until I got to the segfault. Not sure if it helps, but here's the output: http://ix.io/k6j

from mopidy-spotify.

I believe this might have been fixed by jodal/pyspotify@2ed3d2a.

Can you retest?

from mopidy-spotify.

I quickly tested the websocket case and I still get a segfault although it's now slightly different. I have not seen this longer version before.

(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x00007f89f3828a8b in sp_session_starred_create () from /usr/lib/libspotify.so.12
#2  0x00007f89f3b46e6c in _cffi_f_sp_session_starred_create (self=<optimised out>, arg0=<_cffi_backend.CData at remote 0x7f89f2b06c60>) at build/temp.linux-x86_64-2.7/spotify._spotify.c:10979
#3  0x000000000052f936 in PyInt_AsUnsignedLongMask (op=()) at ../Objects/intobject.c:277
#4  0x0000000002484fb0 in ?? ()
#5  0x00007f89f3d8b310 in ?? ()
#6  0x00007f89fa8ab578 in ?? ()
#7  0x00000000004d42be in file_dealloc.13342 (f=0x0) at ../Objects/fileobject.c:625
#8  0x00007f89bc000b50 in ?? ()
#9  0x00007f89f3d6b51c in ?? ()
#10 0x00007f89bc000cc8 in ?? ()
#11 0x0000000002be3f70 in ?? ()
#12 0x00007f89f3d8b110 in ?? ()
#13 0x00007f8900000000 in ?? ()
#14 0x00007f89f3d8b0d0 in ?? ()
#15 0x0000000000000000 in ?? ()

from mopidy-spotify.

jodal/pyspotify@dab0c67 adds another potential fix to this.

Please retest! :-)

from mopidy-spotify.

So far so good!

from mopidy-spotify.

Ok, then we close this :-)

from mopidy-spotify.