tree 0c81ff12275c4a57177819a82460dfb44cba9b4c
parent 038734b1513bfba5dfcd56341eb0bf16649decff
author Bertrand SIMONNET <bsimonnet@chromium.org> 1431556021 -0700
committer Bertrand Simonnet <bsimonnet@chromium.org> 1431645894 +0000

flat_hash: Fix race condition in _setitem.

When setting a value in a flat_hash cache, the pid is used to create a
unique temporary file. If two different processes with the same pid
(different pid namespaces) try to set the cache, the code is racy.

This change converts _setitem to use tempfile which does not rely on
the pid being unique.

BUG=chromium:477727
TEST=trybot run on chromiumos-sdk.

Change-Id: Idee01df326e0408778ad84d7721350c7d7c03ad8
Reviewed-on: https://chromium-review.googlesource.com/270802
Reviewed-by: David James <davidjames@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Tested-by: Bertrand Simonnet <bsimonnet@chromium.org>