Final slash crashes Gloobus-Preview

Bug #426089 reported by leeight
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Gloobus
Fix Released
High
BadChoice

Bug Description

#0 0x08071b59 in gloobusWin::set_title (this=0x85aac00, title=
        {static npos = 4294967295, _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>}, _M_p = 0xbff14e44 "L\bZ\b"}}) at gloobus-preview-window.cpp:95
        pixels_per_char = 6
        max = 0
        width = 260
#1 0x0806ce66 in main (argc=2, argv=0xbff14f04) at gloobus-preview-main.cpp:122
        filename = {static npos = 4294967295,
  _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>},
    _M_p = 0x85a084c "/home/leeight/Dropbox/Photos/"}}
        win3 = (gloobusWin *) 0x85aac00
        w = 320
        h = 380
        title = {static npos = 4294967295,
  _M_dataplus = {<std::allocator<char>> = {<__gnu_cxx::new_allocator<char>> = {<No data fields>}, <No data fields>},
    _M_p = 0x85a084c "/home/leeight/Dropbox/Photos/"}}

Related branches

Revision history for this message
BadChoice (guitarboy000) wrote :

Does it happen always? or just for some files?

Revision history for this message
leeight (leeight) wrote :

always happend

$ ./gloobus-preview ~/Dropbox/Photos/
LOADING PLUGINS:
         /home/leeight/files/gloobus/build/lib/gloobus/audio.so
         /home/leeight/files/gloobus/build/lib/gloobus/comic.so
         /home/leeight/files/gloobus/build/lib/gloobus/folder.so
         /home/leeight/files/gloobus/build/lib/gloobus/icns.so
         /home/leeight/files/gloobus/build/lib/gloobus/openoffice.so
         /home/leeight/files/gloobus/build/lib/gloobus/pdf.so
         /home/leeight/files/gloobus/build/lib/gloobus/pixbuf.so
         /home/leeight/files/gloobus/build/lib/gloobus/text.so
         /home/leeight/files/gloobus/build/lib/gloobus/ttf.so
         /home/leeight/files/gloobus/build/lib/gloobus/video.so
CONTENT FOLDER: /home/leeight/Dropbox/Photos
Mime Type: inode/directory
Creating Plugin Class: list
LIST (iList)
Creating iFolder...

FILE INFO
Screen size: 1280x800
URI: file:///home/leeight/Dropbox/Photos
THUMB: (null)
Type: inode/directory
Size: 19.33 MB

Loading iList...
Scaling image...
No JPG Cover image for this folder
No PNG Cover image for this folder
Image Loaded
Floating point exception (core dumped)

Revision history for this message
BadChoice (guitarboy000) wrote :

Try without the last slash

$ ./gloobus-preview ~/Dropbox/Photos

And tell me if it fails too

Changed in gloobus:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → BadChoice (guitarboy000)
Revision history for this message
leeight (leeight) wrote :

remove the last slash, it works well

thanks for your advice

and one suggestion, i found you define
src/gloobus-preview-defines.h: #define PLUGIN_PATH "/usr/lib/gloobus/"

if i install it to my custom dir, then gloobus will not find those preview plugins.

Revision history for this message
BadChoice (guitarboy000) wrote :

Yes, thats true... I think there's a way that you can define variables depending on the ./configure --prefix but I still don't know it.

Anyway, If you know how it works and want to send me a patch I'll be very greatefull!

Finally, we can change this bug to Last slash crashes gloobus-preview and Fixed plugin directory (As well as data one)

BadChoice (guitarboy000)
summary: - gloobus-preview crashed
+ Final slash crashes Gloobus-Preview
Revision history for this message
Ding Zhou (tualatrix) wrote :

Hi Leeight, nice to see you here. :)

I'm now a member of gloobus-developers and I will help to improve the package related issues.

Your suggestion is really useful, I will try to fix it.

Revision history for this message
BadChoice (guitarboy000) wrote :

I've created the bug about the plugins dir path:

There is info about wich file contains it

https://bugs.launchpad.net/gloobus/+bug/426664

TualatriX: Do you know how to add the translation files with automake? I readed something about and would be great to add this in gloobus-preview (Where it says size, filetype and more thing that can come)

BadChoice (guitarboy000)
Changed in gloobus:
milestone: none → 0.4.1
BadChoice (guitarboy000)
Changed in gloobus:
status: Confirmed → Fix Committed
BadChoice (guitarboy000)
Changed in gloobus:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.