tree 21988471113765f4e9fc277851300bc478240a20
parent 6522aea6060873901308683ba71a23991c624d63
author Sorin Basca <sorinbasca@google.com> 1682707306 +0100
committer Sorin Basca <sorinbasca@google.com> 1683280491 +0100

Fix race condition in java.security.Provider

The supportedFormats and supportedClasses arrays are supposed to
be set only once inside the hasKeyAttributes() function. This is
guarded by the volatile flag hasKeyAttributes, however there was
a race condition when multiple threads check the flag
concurrently and then reach the synchronized block. In that case
the attributes might get set multiple times, concurrent with
reads.

This change fixes this by checking again the guard flag once
inside the synchronized block.

Bug: 276503829
Test: atest CtsLibcoreOjTestCases
Change-Id: Ic544033c1e49e823564c8a05ab3df50286e4c2e0