Ubuntu
sugar-journal-activity package

USB flash mount. Hardcoded UID = 500

Bug #297479 reported by mungewell
4
Affects Status Importance Assigned to Milestone
sugar-journal-activity (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: sugar-journal-activity

It appears that there is a hard coded UID for mounting USB flash drives.

In /usr/share/activities/Journal.activity/volumesmanager.py
--
MOUNT_OPTION_UID = 500
--

This should use the UID of the actual user on the system who is running Sugar. Note: there may be more than one user active in a multiseat/thin client system.
Simon.

Revision history for this message
Morgan Collett (morgan) wrote :

Sugar 0.83 already has a new datastore, which does not yet support USB/removable storage, and the journal has been refactored heavily and included in the sugar shell.

I've logged this bug upstream anyway in the hope that we can patch 0.82.x to fix it.

Bug Watch Updater (bug-watch-updater)
Changed in sugar-journal-activity:
status: Unknown → New
Revision history for this message
mungewell (simon-mungewell) wrote :

Looking at the GIT repo:
http://dev.laptop.org/git?p=journal-activity;a=summary

It looks like the latest release also contains this code/bug.

Given that 'whoami' and 'is it my USB key' becomes a lot more complicated with a multiuser/multiseat is it worth considering ConsoleKit (http://people.freedesktop.org/~mccann/doc/ConsoleKit/ConsoleKit.html) as a possible solution?

ConsoleKit is already in Ubuntu, and I believe it is liked by the Fedora crowd.
Mungewell.

Revision history for this message
Luke Faraone (lfaraone) wrote :

Mungewell,

This seems to be a larger issue that warrents fixing upstream; we don't have the manpower over here to solve it.

Please continue to comment at http://dev.sugarlabs.org/ticket/25

--
Luke Faraone

Revision history for this message
Morgan Collett (morgan) wrote :

Journal has been merged into the 'sugar' component - see http://dev.laptop.org/git?p=sugar;a=tree;f=src/jarabe/journal

The journal-activity component has therefore been dropped from future releases.

Bug Watch Updater (bug-watch-updater)
Changed in sugar-journal-activity:
status: New → Confirmed
Luke Faraone (lfaraone)
Changed in sugar-journal-activity:
status: New → Fix Released
status: Fix Released → Confirmed
Bug Watch Updater (bug-watch-updater)
Changed in sugar-journal-activity:
status: Confirmed → Unknown
Revision history for this message
David Farning (dfarning) wrote :

This has been fixed upstream as of sugar .88

Changed in sugar-journal-activity (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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