Wallch crashes on Mint 17 (Segmentation fault)

Bug #1323574 reported by Joel Cumberland
32
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Wallch
Fix Released
Critical
Alex Solanos

Bug Description

When running Wallch i get a seg fault error (see attachment)

Im running Mint 17 with Wallch version 4.0 installed via apt-get.

Windows Manager: Cinnamon 2.2.13

Thanks,
Joel

Revision history for this message
Joel Cumberland (joel-c-c) wrote :
description: updated
Revision history for this message
Alex Solanos (hakermania) wrote :

Thanks for your bug report!

Please provide us with steps in order to reproduce the problem.

Revision history for this message
Joel Cumberland (joel-c-c) wrote :

I open a terminal session .e.g gnome terminal and I've check the var DISPLAY is set correctly .e.g DISPLAY=:0 (so it shouldn't be anything to do with the display) and I run the command wallch as regular user. I did notice when running wallch --change=DIR it does change the wallpaper, but I cant open the GUI as I get a Seq fault as shown in attachment.

Could it be something to do with my Qt libs ? I ran wallch v3 on my Mint 16 system with no troubles.

If you need any more info I'm happy to provide.

Thanks

Revision history for this message
Alex Solanos (hakermania) wrote :

I'm not sure what's the problem here. I will try to test Wallch 4.0 in Mint 17 MATE desktop and I will let you know!

Changed in wallpaper-changer:
assignee: nobody → Alex Solanos (hakermania)
Changed in wallpaper-changer:
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
Alex Solanos (hakermania) wrote :

This has something to do with Wallch's indicator, it seems.

I am not sure who is to blame here, because the same code runs just fine under Ubuntu 14.04 (indicator is created and responds to actions just fine).

I will investigate it further.

Revision history for this message
Andreas M (amuttsch) wrote :

Same problem here. Mint 17 x64 with Cinnamon.

Is there a guide on how to compile the source code, so I can try to get a proper stack trace and try to help investigating this issue?
qmake shows the following message:

Project ERROR: Package appindicator-0.1 not found

appindicator is installed.

Revision history for this message
Alex Solanos (hakermania) wrote :

Thanks for trying to resolve the problem.

In order to build wallch you indeed will use 'qmake', but first make sure to install all the packages listed on the Build-Depends section of debian/control file.

The latest Wallch source can be obtained using bzr branch lp:wallpaper-changer

I just want to say that I believe that it may not be Wallch to blame as the same code runs perfectly fine in Ubuntu 14.04 LTS and maybe Mint's 17 indicator drawer isn't completely stable yet.

Revision history for this message
Daniel Piovezan (danielbzd) wrote :

Hi, I too am having that problem with Mint 17 Cinnamon. I'll attach a crash log.

But here's something: I'm having that problem with wallch 4.0-0ubuntu4, from the Ubuntu repos. I tried installing one from this PPA: https://launchpad.net/~wallch/+archive/wallch-4.0/+packages which is called 4.0-1trusty, but it complains of a missing depency, libqt5core5. I can't find that package with synaptic, but I have libkt5core5a installed.

Revision history for this message
Alex Solanos (hakermania) wrote :

Thanks, Daniel!

Try installing this PPA instead: https://launchpad.net/~hakermania/+archive/wallch-test

The PPA you tried to download from was the one used to push Wallch into the repos. During this process the package got remained. The maintainers noticed the change and fixed it but the PPA remained unchanged. That's why you got an error.

Notice that aside from that small change to that package name, there is no other difference between the version that you tried to install and the version inside the Ubuntu repositories.

On the other hand, the PPA I listed above (wallch-test) contains bug fixes of the release included into the softwarece center, but it is still on testing phase.

Revision history for this message
Michael Webster (miketwebster) wrote :

So I think this may be something silly...

We recently registered with freedesktop.org to get "Cinnamon" listed as a valid desktop (for many checks - app visibility in menus, startup programs, etc..)

It's been accepted, but in the meantime, according to the standard for desktop names we thought we were supposed to prefix X- if it's an unregistered name: http://standards.freedesktop.org/menu-spec/menu-spec-1.0.html#onlyshowin-registry. Upon a quick re-reading, desktop names were not specifically included in this X- exception (it could be mentioned elsewhere, however, just did a quick re-read.

Regardless, running wallch by:

XDG_CURRENT_DESKTOP=GNOME wallch

Works fine, at least in my limited testing. I didn't dig into this code much, and the breakpoint where it segfaults wasn't very informative.

Without knowing more, for the time being, until Cinnamon rolls out as a registered desktop name, I would maybe just supply an additional .desktop file with:

OnlyShowIn=X-Cinnamon
Exec=XDG_CURRENT_DESKTOP=GNOME wallch

and NotShowIn=X-Cinnamon in your other .desktop file.

This problem will probably eventually go away.

Revision history for this message
Michael Webster (miketwebster) wrote :

Ah scratch that, it makes the Exec line invalid. I'll give some thought, and maybe you could just check it and change on the fly at startup, either for X-Cinnamon specifically, or for anything not currently registered (Since the result is the same if I set my desktop name to "FOOBAR"

Changed in wallpaper-changer:
status: Confirmed → Fix Committed
Revision history for this message
Alex Solanos (hakermania) wrote :

That was not Wallch's fault, apparently, but the workaround is 2 lines of code, so I uploaded it to the trunk.

The fix will be released along with the next version of Wallch.

Changed in wallpaper-changer:
status: Fix Committed → Fix Released
Revision history for this message
Alin Andrei (nilarimogard) wrote :

It's still not fixed - tested with Wallch 4.11-1trusty from this PPA: https://launchpad.net/~wallch/+archive/wallch-4.0

This works though: "XDG_CURRENT_DESKTOP=GNOME wallch"

Revision history for this message
Alex Solanos (hakermania) wrote :

Thanks a lot for the comment Alin.

Basically what I did was to check whether the XDG_CURRENT_DESKTOP is set to "default.desktop" and if it is, then set it to "gnome".

That worked in my case for Mint17, so I guess that the value of XDG_CURRENT_DESKTOP is different in your environment. This is quite weird as we are both running Mint17, I suppose.

Can you please post the output of echo $XDG_CURRENT_DESKTOP ?

Thanks a lot again.

Revision history for this message
Alin Andrei (nilarimogard) wrote :

Firstly, note that I'm using Mint 17 in Virtualbox. Also, I've installed Mint 17 RC (but all packages are up to date) so maybe that has something to do with the bug?

Here's the output:
$ echo $XDG_CURRENT_DESKTOP
X-Cinnamon

Revision history for this message
Alex Solanos (hakermania) wrote :

Yep, that explains the issue. My output was just "default.desktop". I guess I will immediately upload a fix that contains X-Cinnamon as well :)

Revision history for this message
Alin Andrei (nilarimogard) wrote :

Great, thank you!

Revision history for this message
Alex Solanos (hakermania) wrote :

The new version is available through the new PPA. Can you confirm that it now works for you?

Revision history for this message
Alex Solanos (hakermania) wrote :

through the same* PPA

Revision history for this message
Alin Andrei (nilarimogard) wrote :

I've tried it and it works! Thank you.

Revision history for this message
Chugajstyr (chugajstyr) wrote :

I

Revision history for this message
Chugajstyr (chugajstyr) wrote :

I`am using Mint 17 too (DE - Mate 1.8). Even latest Wallch v4.12 from ppa:wallch/wallch-4.0 crashed on startup.

P.S. Sorry for previous stupid comment, something wrong with touchpad...

Revision history for this message
Alex Solanos (hakermania) wrote :

Hello ColdEgo! I am starting to think that everyone has a different value fro XDG_CURRENT_DESKTOP under Cinnamon!

Can you please open a terminal window and post here the output of

echo $XDG_CURRENT_DESKTOP ?

I also feel the urge to repeat that this is not Wallch's fault but Cinnamon's, for using an unregistered desktop name for XDG_CURRENT_DESKTOP (as the devs themselves told me via IRC)

Revision history for this message
Michael Webster (miketwebster) wrote :

He's not running Cinnamon, he's running Mate. Everyone running Cinnamon 2.2 uses the same XDG_CURRENT_DESKTOP value.

I never said it was wallch's fault, but it's possibly something in libnotify. However, this is the only program I've seen thus far that had any issue with an unregistered name.

Revision history for this message
Alex Solanos (hakermania) wrote :

On the other hand, Mint is the only environment that causes Wallch to crash :)

I did not notice that he was running Mate, though.

I guess it has something to do with libappindicator. What makes you think that is has to do with libnotify?

Revision history for this message
Michael Webster (miketwebster) wrote :

Sorry I meant libappindicator.

summary: - Wallch crashes (Segmentation fault)
+ Wallch crashes on Mint 17 (Segmentation fault)
Revision history for this message
Chugajstyr (chugajstyr) wrote :

If it somehow helps, the output of command

"echo $XDG_CURRENT_DESKTOP" in my terminal was

"default".

Revision history for this message
Alex Solanos (hakermania) wrote :

Thanks for the information ColdEgo. Please see if the updated 4.14 version (from the same PPA) works now for you.

Revision history for this message
Chugajstyr (chugajstyr) wrote :

After latest update from repo today - the program start successfully. Thanks, it was fast bugfixing, I`am really impressed! Will there be also updated version in the official repositories?

BTW, a little note - updater show me program version as "4.14-1trusty", and in "About" I see "Wallch 4.12", but actually it does not matter.

Revision history for this message
Alex Solanos (hakermania) wrote :

Glad it fixed it for you.

Don't worry about the "4.12" version on the About dialog. I just changed to change the in-program version.

As for a new version on the official repos, we intend to but not yet.

Revision history for this message
Giannis Tsagatakis (jtsagata) wrote :

Seems there is issues with the mint 16 Xfce also. Check: http://linuxmint.gr/index.php/topic,3060.msg25964/topicseen.html#new
(Sorry for the greek link, but the developers will be able to read it)

Revision history for this message
Alex Solanos (hakermania) wrote :

Hello Giannis πατριώτη! :)

Please do me a favor and open a new bug report for this, because this one is for Mint 17 specifically.

Also, it would be nice to include what the bug is and the steps in order to reproduce it, along with the link that you provide.

Thanks!

Revision history for this message
Giannis Tsagatakis (jtsagata) wrote :

Hello Alex,
I was mistaken and it effects mint 17 Xfce. Sorry for that. And it doesn't affect me. I just report it. The user that have done the report is a newbie. If you like, in order not to play the "brocken phone game" here, assist him directly to get the bug report in that forum.

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.