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

HADOOP-12296. when setnetgrent returns 0 in linux, exception should be thrown. Contributed by Chang Li 

(cherry picked from commit b9d25c3ee2)
This commit is contained in:
Jason Lowe 2015-11-04 16:34:01 +00:00
parent 8334d86472
commit b9389ac5b5
2 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hadoop-common-project/hadoop-common/CHANGES.txt
View File

@ -803,6 +803,9 @@ Release 2.7.3 - UNRELEASED
BUG FIXES
HADOOP-12296. when setnetgrent returns 0 in linux, exception should be
thrown (Chang Li via jlowe)
Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES

18
hadoop-common-project/hadoop-common/src/main/native/src/org/apache/hadoop/security/JniBasedUnixGroupsNetgroupMapping.c
View File

@ -57,6 +57,7 @@ Java_org_apache_hadoop_security_JniBasedUnixGroupsNetgroupMapping_getUsersForNet
int setnetgrentCalledFlag = 0;
// if not NULL then THROW exception
char *errorType = NULL;
char *errorMessage = NULL;
cgroup = (*env)->GetStringUTFChars(env, jgroup, NULL);
@ -94,7 +95,14 @@ Java_org_apache_hadoop_security_JniBasedUnixGroupsNetgroupMapping_getUsersForNet
}
}
}
#if defined(__linux__)
else {
errorType = "java/io/IOException";
errorMessage =
"no netgroup of this name is known or some other error occurred";
goto END;
}
#endif
//--------------------------------------------------
// build return data (java array)
@ -103,7 +111,7 @@ Java_org_apache_hadoop_security_JniBasedUnixGroupsNetgroupMapping_getUsersForNet
(*env)->FindClass(env, "java/lang/String"),
NULL);
if (jusers == NULL) {
errorMessage = "java/lang/OutOfMemoryError";
errorType = "java/lang/OutOfMemoryError";
goto END;
}
@ -114,7 +122,7 @@ Java_org_apache_hadoop_security_JniBasedUnixGroupsNetgroupMapping_getUsersForNet
for(current = userListHead; current != NULL; current = current->next) {
jstring juser = (*env)->NewStringUTF(env, current->string);
if (juser == NULL) {
errorMessage = "java/lang/OutOfMemoryError";
errorType = "java/lang/OutOfMemoryError";
goto END;
}
(*env)->SetObjectArrayElement(env, jusers, i++, juser);
@ -134,8 +142,8 @@ END:
}
// return results or THROW
if(errorMessage) {
THROW(env, errorMessage, NULL);
if(errorType) {
THROW(env, errorType, errorMessage);
return NULL;
} else {
return jusers;