Unity Chromium Bookmarks Scope

Code review comment for lp:~davidc3/unity-scope-chromiumbookmarks/icons-n-dups

  1. icons-n-dups
  2. Merge into trunk
Revision history for this message
Paweł Stołowski (stolowski) wrote :

48 + open('/tmp/unity-scope-chromiumbookmarks/' + thumbname , 'wb').write(imageinfo)
49 + icon = '/tmp/unity-scope-chromiumbookmarks/' + thumbname

This doesn't seem to be multi-session friendly and also can easily be abused on a multi-user system. Can you generate a true temporary name on startup?

review: Needs Fixing

« Back to merge proposal