Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(263)

Issue 17415: Download into unique subdir of /tmp to avoid permission problems (e.g. root n... (Closed)

Created:
11 years, 11 months ago by dank
Modified:
9 years, 7 months ago
Reviewers:
Dean McNamee
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Download into unique subdir of /tmp to avoid permission problems (e.g. root not allowed to access nfs). Clean up carefully afterwards. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=7728

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
M build/install-build-deps.sh View 2 chunks +12 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
dank
How's this look?
11 years, 11 months ago (2009-01-08 13:48:27 UTC) #1
Dean McNamee
On 2009/01/08 13:48:27, dank wrote: > How's this look? Does this still avoid umask problems? ...
11 years, 11 months ago (2009-01-08 14:28:04 UTC) #2
Dean McNamee
11 years, 11 months ago (2009-01-08 14:29:25 UTC) #3
On 2009/01/08 14:28:04, Dean McNamee wrote:
> On 2009/01/08 13:48:27, dank wrote:
> > How's this look?
> 
> Does this still avoid umask problems?  Will the perms via sudo always be able
to
> access your own files if the umask doesn't have g+ or o+?

Blah, root should be able to read anything, sorry I'm dumb.  It must have been
NFS.  LGTM!

Powered by Google App Engine
This is Rietveld 408576698