tree 8a848cb83007a0ba2a67b6f8e86fd39237287863
parent 59752a96e10313d3e6b540a5cb77881fb30cb9a3
author Chris Masone <cmasone@chromium.org> 1401299559 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> 1401321543 +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
Reviewed-on: https://chromium-review.googlesource.com/201831
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Chris Masone <cmasone@chromium.org>
Tested-by: Chris Masone <cmasone@chromium.org>