Bug #14793

Custom Desktop launchers are totally buggy

Added by intrigeri 2017-10-06 05:32:23 . Updated 2017-11-15 11:33:46 .

Status:
Resolved
Priority:
High
Assignee:
Category:
Target version:
Start date:
2017-10-06
Due date:
% Done:

100%

Feature Branch:
bugfix/14783-trust-desktop-launchers
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

They get the default icon instead of the configured one, e.g. tails-documentation.desktop specifies Icon=/usr/share/icons/gnome/48x48/categories/system-help.png, which is installed, but a text file -looking icon is displayed instead.

Their Name= is ignored i.e. the displayed label is e.g. tails-documentation.desktop.

Double-clicking on the icon opens an “Untrusted application launcher” dialog.

All these are caused by the fix for Bug #14584, that will land in Stretch any time now, so it needs a fix based on our stable branch: if I click “Trust and launch” then the icon and name get fixed and I see no confirmation dialog next time I double-click the same launcher.

I think the fix is to add a trusted attribute in a gio/gvfs metadata, which is stored on the filesystem in ~/.local/share/gvfs-metadata.

I’m not surprised this component is bitrotting, see Bug #13461#note-13 and Feature #11717#note-13.


Subtasks


Related issues

Related to Tails - Bug #13461: The Desktop icons are sometimes not displayed since the upgrade to Stretch Resolved 2017-07-12
Related to Tails - Feature #11717: Drop launchers from the Desktop Rejected 2016-08-25
Related to Tails - Bug #14584: Backport security fix for Nautilus .desktop file RCE Rejected 2017-09-01
Related to Tails - Bug #16283: No more icons on desktop in buster Resolved 2019-01-05

History

#1 Updated by intrigeri 2017-10-06 05:32:46

  • related to Bug #13461: The Desktop icons are sometimes not displayed since the upgrade to Stretch added

#2 Updated by intrigeri 2017-10-06 05:33:00

#3 Updated by intrigeri 2017-10-06 05:33:59

  • Subject changed from Custom Desktop launchers get the default icon instead of the configured one on Buster to Custom Desktop launchers get the default icon instead of the configured one on Buster and their Name= is ignored

#4 Updated by intrigeri 2017-10-06 05:39:11

  • Subject changed from Custom Desktop launchers get the default icon instead of the configured one on Buster and their Name= is ignored to Custom Desktop launchers are totally buggy on Buster
  • Description updated
  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 10
  • Feature Branch set to feature/buster

#5 Updated by intrigeri 2017-10-06 05:39:27

  • related to Bug #14584: Backport security fix for Nautilus .desktop file RCE added

#6 Updated by intrigeri 2017-10-06 05:40:49

  • Priority changed from Normal to High

High prio as it totally breaks our test suite. We can downgrade priority if a short-term workaround is implemented there.

#7 Updated by intrigeri 2017-10-06 05:42:29

I think we can use gio set to set the needed attribute.

#8 Updated by intrigeri 2017-10-07 11:47:25

  • Status changed from In Progress to Resolved
  • % Done changed from 10 to 100

#9 Updated by intrigeri 2017-10-09 14:37:55

  • Subject changed from Custom Desktop launchers are totally buggy on Buster to Custom Desktop launchers are totally buggy
  • Status changed from Resolved to In Progress
  • Assignee changed from intrigeri to anonym
  • Target version changed from Tails_4.0 to Tails_3.3
  • % Done changed from 100 to 50
  • QA Check set to Ready for QA
  • Feature Branch changed from feature/buster to bugfix/14783-trust-desktop-launchers
  • Type of work changed from Research to Code

#10 Updated by intrigeri 2017-10-09 14:43:11

I don’t remember if I’ve tested this branch on Stretch. And for sure I didn’t do that after the Nautilus security update that makes us want this in stable was available. I’m confident it should work but if that’s too sloppy for you, I would understand => reassign to me.

#11 Updated by anonym 2017-10-09 16:23:17

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

intrigeri wrote:
> I don’t remember if I’ve tested this branch on Stretch. And for sure I didn’t do that after the Nautilus security update that makes us want this in stable was available. I’m confident it should work but if that’s too sloppy for you, I would understand => reassign to me.

Nope, it’s fine. I built stable with this branch merged and it worked fine => merged!

#12 Updated by anonym 2017-11-15 11:33:46

  • Status changed from Fix committed to Resolved

#13 Updated by hefee 2019-01-07 16:47:39

  • related to Bug #16283: No more icons on desktop in buster added