-
bartfab@chromium.org authored
Speculative revert as a lot of Mac bots broke when this landed: http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%282%29/builds/54365 http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%284%29/builds/43947 http://build.chromium.org/p/chromium.mac/builders/Mac%2010.6%20Tests%20%28dbg%29%283%29/builds/52314 > Fix Mac sandbox meta data access > > Sandbox::AllowMetadataForPath() currently allow all metadata access due to > https://codereview.chromium.org/10539009/ made the for loop comparison > in Sandbox::AllowMetadataForPath() always false, when we actually only > want to allow access to the path and all its parent path until root. > > Turn the for loop to a do/while loop instead as it's a better fit, also > add a test case for Sandbox::AllowMetadataForPath(). > > It should only affect component builds on OS X 10.6 and utility process > as no other process is using this mechanism. > > Review URL: https://codereview.chromium.org/472513002 TBR=jiangj@opera.com Review URL: https://codereview.chromium.org/469293002 Cr-Commit-Position: refs/heads/master@{#289541} git-svn-id: svn://svn.chromium.org/chrome/trunk/src@289541 0039d316-1c4b-4281-b951-d872f2087c98
2db07c1a
