HDFS-6411. nfs-hdfs-gateway mount raises I/O error and hangs when a unauthorized user attempts to access it. Contributed by Brandon Li
git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/trunk@1597895 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
1c867b1de8
commit
0ec6fc9e3c
|
@ -46,10 +46,12 @@ public class ACCESS3Response extends NFS3Response {
|
|||
@Override
|
||||
public XDR writeHeaderAndResponse(XDR out, int xid, Verifier verifier) {
|
||||
super.writeHeaderAndResponse(out, xid, verifier);
|
||||
out.writeBoolean(true);
|
||||
postOpAttr.serialize(out);
|
||||
if (this.getStatus() == Nfs3Status.NFS3_OK) {
|
||||
out.writeBoolean(true);
|
||||
postOpAttr.serialize(out);
|
||||
out.writeInt(access);
|
||||
} else {
|
||||
out.writeBoolean(false);
|
||||
}
|
||||
return out;
|
||||
}
|
||||
|
|
|
@ -124,6 +124,7 @@ import org.apache.hadoop.oncrpc.security.VerifierNone;
|
|||
import org.apache.hadoop.security.AccessControlException;
|
||||
import org.apache.hadoop.security.SecurityUtil;
|
||||
import org.apache.hadoop.security.UserGroupInformation;
|
||||
import org.apache.hadoop.security.authorize.AuthorizationException;
|
||||
import org.jboss.netty.buffer.ChannelBuffer;
|
||||
import org.jboss.netty.buffer.ChannelBuffers;
|
||||
import org.jboss.netty.channel.Channel;
|
||||
|
@ -270,6 +271,17 @@ public class RpcProgramNfs3 extends RpcProgram implements Nfs3Interface {
|
|||
Nfs3FileAttributes attrs = null;
|
||||
try {
|
||||
attrs = writeManager.getFileAttr(dfsClient, handle, iug);
|
||||
} catch (RemoteException r) {
|
||||
LOG.warn("Exception ", r);
|
||||
IOException io = r.unwrapRemoteException();
|
||||
/**
|
||||
* AuthorizationException can be thrown if the user can't be proxy'ed.
|
||||
*/
|
||||
if (io instanceof AuthorizationException) {
|
||||
return new GETATTR3Response(Nfs3Status.NFS3ERR_ACCES);
|
||||
} else {
|
||||
return new GETATTR3Response(Nfs3Status.NFS3ERR_IO);
|
||||
}
|
||||
} catch (IOException e) {
|
||||
LOG.info("Can't get file attribute, fileId=" + handle.getFileId(), e);
|
||||
response.setStatus(Nfs3Status.NFS3ERR_IO);
|
||||
|
@ -499,6 +511,17 @@ public class RpcProgramNfs3 extends RpcProgram implements Nfs3Interface {
|
|||
securityHandler.getUid(), securityHandler.getGid(), attrs);
|
||||
|
||||
return new ACCESS3Response(Nfs3Status.NFS3_OK, attrs, access);
|
||||
} catch (RemoteException r) {
|
||||
LOG.warn("Exception ", r);
|
||||
IOException io = r.unwrapRemoteException();
|
||||
/**
|
||||
* AuthorizationException can be thrown if the user can't be proxy'ed.
|
||||
*/
|
||||
if (io instanceof AuthorizationException) {
|
||||
return new ACCESS3Response(Nfs3Status.NFS3ERR_ACCES);
|
||||
} else {
|
||||
return new ACCESS3Response(Nfs3Status.NFS3ERR_IO);
|
||||
}
|
||||
} catch (IOException e) {
|
||||
LOG.warn("Exception ", e);
|
||||
return new ACCESS3Response(Nfs3Status.NFS3ERR_IO);
|
||||
|
@ -1680,6 +1703,17 @@ public class RpcProgramNfs3 extends RpcProgram implements Nfs3Interface {
|
|||
|
||||
return new FSSTAT3Response(Nfs3Status.NFS3_OK, attrs, totalBytes,
|
||||
freeBytes, freeBytes, maxFsObjects, maxFsObjects, maxFsObjects, 0);
|
||||
} catch (RemoteException r) {
|
||||
LOG.warn("Exception ", r);
|
||||
IOException io = r.unwrapRemoteException();
|
||||
/**
|
||||
* AuthorizationException can be thrown if the user can't be proxy'ed.
|
||||
*/
|
||||
if (io instanceof AuthorizationException) {
|
||||
return new FSSTAT3Response(Nfs3Status.NFS3ERR_ACCES);
|
||||
} else {
|
||||
return new FSSTAT3Response(Nfs3Status.NFS3ERR_IO);
|
||||
}
|
||||
} catch (IOException e) {
|
||||
LOG.warn("Exception ", e);
|
||||
return new FSSTAT3Response(Nfs3Status.NFS3ERR_IO);
|
||||
|
|
|
@ -681,6 +681,9 @@ Release 2.4.1 - UNRELEASED
|
|||
HDFS-6397. NN shows inconsistent value in deadnode count.
|
||||
(Mohammad Kamrul Islam via kihwal)
|
||||
|
||||
HDFS-6411. nfs-hdfs-gateway mount raises I/O error and hangs when a
|
||||
unauthorized user attempts to access it (brandonli)
|
||||
|
||||
Release 2.4.0 - 2014-04-07
|
||||
|
||||
INCOMPATIBLE CHANGES
|
||||
|
|
Loading…
Reference in New Issue