New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 672866 link

Starred by 1 user
Status: Fixed
Owner:
aga...@chromium.org
Closed: Dec 2016
Cc:
serg...@chromium.org
█ andyb...@chromium.org
rmis...@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----


Previous locations:
gerrit:5106


Sign in to add a comment

User should not be able to upload patchsets to someone else's CL

Project Member Reported by serg...@chromium.org, Dec 9 2016 
Example CL: https://chromium-review.googlesource.com/c/410201/
I've uploaded patchset 7 killing all files in the original CL.
This was a mistake, but someone may misuse this for malicious purposes.

Comment 1 by andyb...@chromium.org, Dec 9 2016

Labels: -Restrict-View-Google

Comment 2 by andyb...@chromium.org, Dec 9 2016

Project: chromium
Moved issue gerrit:5106 to now be  issue chromium:672866 .

Comment 3 by andyb...@chromium.org, Dec 9 2016

Components: Infra>Codereview>Gerrit
Labels: Milestone-Dogfood Proj-Gerrit-Migration Pri-2
Owner: aga...@chromium.org
Status: Assigned (was: New)
Looks like we need to configure the infra project not to allow pushing patchsets to others’ changes.

Comment 4 by aga...@chromium.org, Dec 9 2016

Cc: andyb...@chromium.org serg...@chromium.org
Status: Fixed (was: Assigned)
Only committers can upload patchsets to other people's CLs. sergiyb is a committer.
►

Sign in to add a comment