tree 5fa8b5b4b1a2b8501fb83485af6ce46a674fb560
parent c22f2840aaa15598cf3548c0cf018116e1faf5f1
author Chris Masone <cmasone@chromium.org> 1401299559 -0700
committer Mike Frysinger <vapier@chromium.org> 1402336981 +0000

Remove race in enewuser, enewgroup

We had an obvious TOCTOU problem in enewuser and enewgroup.
The code would check to see if a user or group was already
created early in these functions, then do a bunch of work
and check again. If the account was created in the meantime,
it'd barf on the second check. This check is redundant anyway
so remove it.

BUG=chromium:378270
TEST=remove accounts for a package and emerge it again
TEST=release trybot

Change-Id: If522df57468cd79466c27945b2240d9bc11ca503
Previous-Reviewed-on: https://chromium-review.googlesource.com/201831
(cherry picked from commit c6778c492f5e75819c316e75f2aae44f33534e4e)
Reviewed-on: https://chromium-review.googlesource.com/202923
Reviewed-by: Chris Masone <cmasone@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>