HADOOP-17858. Avoid possible class loading deadlock with VerifierNone initialization (#3321)
(cherry picked from commit fc566ad9b0
)
This commit is contained in:
parent
224b42108d
commit
fc6b1cafd4
|
@ -24,7 +24,6 @@ import java.net.SocketAddress;
|
||||||
|
|
||||||
import org.apache.hadoop.thirdparty.com.google.common.annotations.VisibleForTesting;
|
import org.apache.hadoop.thirdparty.com.google.common.annotations.VisibleForTesting;
|
||||||
import org.apache.hadoop.oncrpc.RpcAcceptedReply.AcceptState;
|
import org.apache.hadoop.oncrpc.RpcAcceptedReply.AcceptState;
|
||||||
import org.apache.hadoop.oncrpc.security.Verifier;
|
|
||||||
import org.apache.hadoop.oncrpc.security.VerifierNone;
|
import org.apache.hadoop.oncrpc.security.VerifierNone;
|
||||||
import org.apache.hadoop.portmap.PortmapMapping;
|
import org.apache.hadoop.portmap.PortmapMapping;
|
||||||
import org.apache.hadoop.portmap.PortmapRequest;
|
import org.apache.hadoop.portmap.PortmapRequest;
|
||||||
|
@ -214,7 +213,7 @@ public abstract class RpcProgram extends SimpleChannelUpstreamHandler {
|
||||||
private void sendAcceptedReply(RpcCall call, SocketAddress remoteAddress,
|
private void sendAcceptedReply(RpcCall call, SocketAddress remoteAddress,
|
||||||
AcceptState acceptState, ChannelHandlerContext ctx) {
|
AcceptState acceptState, ChannelHandlerContext ctx) {
|
||||||
RpcAcceptedReply reply = RpcAcceptedReply.getInstance(call.getXid(),
|
RpcAcceptedReply reply = RpcAcceptedReply.getInstance(call.getXid(),
|
||||||
acceptState, Verifier.VERIFIER_NONE);
|
acceptState, VerifierNone.INSTANCE);
|
||||||
|
|
||||||
XDR out = new XDR();
|
XDR out = new XDR();
|
||||||
reply.write(out);
|
reply.write(out);
|
||||||
|
|
|
@ -27,8 +27,6 @@ import org.apache.hadoop.oncrpc.XDR;
|
||||||
*/
|
*/
|
||||||
public abstract class Verifier extends RpcAuthInfo {
|
public abstract class Verifier extends RpcAuthInfo {
|
||||||
|
|
||||||
public static final Verifier VERIFIER_NONE = new VerifierNone();
|
|
||||||
|
|
||||||
protected Verifier(AuthFlavor flavor) {
|
protected Verifier(AuthFlavor flavor) {
|
||||||
super(flavor);
|
super(flavor);
|
||||||
}
|
}
|
||||||
|
|
|
@ -24,6 +24,8 @@ import org.apache.hadoop.thirdparty.com.google.common.base.Preconditions;
|
||||||
/** Verifier used by AUTH_NONE. */
|
/** Verifier used by AUTH_NONE. */
|
||||||
public class VerifierNone extends Verifier {
|
public class VerifierNone extends Verifier {
|
||||||
|
|
||||||
|
public static final Verifier INSTANCE = new VerifierNone();
|
||||||
|
|
||||||
public VerifierNone() {
|
public VerifierNone() {
|
||||||
super(AuthFlavor.AUTH_NONE);
|
super(AuthFlavor.AUTH_NONE);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue