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

HBASE-12774 Fix the inconsistent permission checks for bulkloading. (Srikanth Srungarapu)

Browse Source
This commit is contained in:
Enis Soztutar 2015-01-21 17:03:55 -08:00
parent 08cf057c6b
commit ba2d05e148
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
View File

@ -1936,7 +1936,7 @@ public class AccessController extends BaseMasterAndRegionObserver
}
/**
* Verifies user has WRITE privileges on
* Verifies user has CREATE privileges on
* the Column Families involved in the bulkLoadHFile
* request. Specific Column Write privileges are presently
* ignored.
@ -1993,11 +1993,11 @@ public class AccessController extends BaseMasterAndRegionObserver
PrepareBulkLoadRequest request) throws IOException {
RegionCoprocessorEnvironment e = ctx.getEnvironment();
AuthResult authResult = hasSomeAccess(e, "prePrepareBulkLoad", Action.WRITE);
AuthResult authResult = hasSomeAccess(e, "prePrepareBulkLoad", Action.CREATE);
logResult(authResult);
if (!authResult.isAllowed()) {
throw new AccessDeniedException("Insufficient permissions (table=" +
e.getRegion().getTableDesc().getTableName() + ", action=WRITE)");
e.getRegion().getTableDesc().getTableName() + ", action=CREATE)");
}
}
@ -2013,11 +2013,11 @@ public class AccessController extends BaseMasterAndRegionObserver
CleanupBulkLoadRequest request) throws IOException {
RegionCoprocessorEnvironment e = ctx.getEnvironment();
AuthResult authResult = hasSomeAccess(e, "preCleanupBulkLoad", Action.WRITE);
AuthResult authResult = hasSomeAccess(e, "preCleanupBulkLoad", Action.CREATE);
logResult(authResult);
if (!authResult.isAllowed()) {
throw new AccessDeniedException("Insufficient permissions (table=" +
e.getRegion().getTableDesc().getTableName() + ", action=WRITE)");
e.getRegion().getTableDesc().getTableName() + ", action=CREATE)");
}
}