Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-14195. CredentialProviderFactory$getProviders is not thread-safe. Contributed by Vihang Karajgaonkar. 

(cherry picked from commit 128015584d)
(cherry picked from commit a92f654aa2)
This commit is contained in:
John Zhuge 2017-04-12 20:02:39 -07:00
parent 7a3f9e8fac
commit fac5bdc023
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/CredentialProviderFactory.java
View File

@ -54,7 +54,10 @@ public abstract class CredentialProviderFactory {
try { try {
URI uri = new URI(path); URI uri = new URI(path);
boolean found = false; boolean found = false;
for(CredentialProviderFactory factory: serviceLoader) { // Iterate serviceLoader in a synchronized block since
// serviceLoader iterator is not thread-safe.
synchronized (serviceLoader) {
for (CredentialProviderFactory factory : serviceLoader) {
CredentialProvider kp = factory.createProvider(uri, conf); CredentialProvider kp = factory.createProvider(uri, conf);
if (kp != null) { if (kp != null) {
result.add(kp); result.add(kp);
@ -62,6 +65,7 @@ public abstract class CredentialProviderFactory {
break; break;
} }
} }
}
if (!found) { if (!found) {
throw new IOException("No CredentialProviderFactory for " + uri + " in " + throw new IOException("No CredentialProviderFactory for " + uri + " in " +
CREDENTIAL_PROVIDER_PATH); CREDENTIAL_PROVIDER_PATH);