You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by pl...@apache.org on 2015/11/26 03:12:03 UTC
directory-kerby git commit: Revert "DIRKRB-475 The decoded asn1
object with the wrong tagFlags."
Repository: directory-kerby
Updated Branches:
refs/heads/master 5706da2eb -> e69ab4d5e
Revert "DIRKRB-475 The decoded asn1 object with the wrong tagFlags."
This reverts commit 5706da2ebb7cc5aea15dcb186efe2995de28f6af.
Project: http://git-wip-us.apache.org/repos/asf/directory-kerby/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-kerby/commit/e69ab4d5
Tree: http://git-wip-us.apache.org/repos/asf/directory-kerby/tree/e69ab4d5
Diff: http://git-wip-us.apache.org/repos/asf/directory-kerby/diff/e69ab4d5
Branch: refs/heads/master
Commit: e69ab4d5e6df87a9b0fd13743ff09f26b1be1311
Parents: 5706da2
Author: plusplusjiajia <ji...@intel.com>
Authored: Thu Nov 26 10:18:28 2015 +0800
Committer: plusplusjiajia <ji...@intel.com>
Committed: Thu Nov 26 10:18:28 2015 +0800
----------------------------------------------------------------------
.../main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java | 3 +--
.../java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java | 4 +---
.../src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java | 3 +--
3 files changed, 3 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/e69ab4d5/kerby-asn1/src/main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java
----------------------------------------------------------------------
diff --git a/kerby-asn1/src/main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java b/kerby-asn1/src/main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java
index 305f986..f0fbf35 100644
--- a/kerby-asn1/src/main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java
+++ b/kerby-asn1/src/main/java/org/apache/kerby/asn1/type/AbstractAsn1Type.java
@@ -209,10 +209,9 @@ public abstract class AbstractAsn1Type<T> implements Asn1Type {
protected void decode(LimitedByteBuffer content) throws IOException {
int tag = readTag(content);
int tagNo = readTagNo(content, tag);
- int tagFlags = tag & 0xe0;
int length = readLength(content);
- decode(tagNo, tagNo, new LimitedByteBuffer(content, length));
+ decode(tag, tagNo, new LimitedByteBuffer(content, length));
}
public void decode(int tagFlags, int tagNo,
http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/e69ab4d5/kerby-kerb/kerb-core-test/src/test/java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java
----------------------------------------------------------------------
diff --git a/kerby-kerb/kerb-core-test/src/test/java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java b/kerby-kerb/kerb-core-test/src/test/java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java
index 8c005b4..da6c7ce 100644
--- a/kerby-kerb/kerb-core-test/src/test/java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java
+++ b/kerby-kerb/kerb-core-test/src/test/java/org/apache/kerby/kerberos/kerb/codec/CodecTest.java
@@ -19,8 +19,8 @@
*/
package org.apache.kerby.kerberos.kerb.codec;
-import org.apache.kerby.kerberos.kerb.KrbCodec;
import org.apache.kerby.kerberos.kerb.KrbException;
+import org.apache.kerby.kerberos.kerb.KrbCodec;
import org.apache.kerby.kerberos.kerb.spec.base.CheckSum;
import org.apache.kerby.kerberos.kerb.spec.base.CheckSumType;
import org.junit.Test;
@@ -41,7 +41,5 @@ public class CodecTest {
assertThat(restored).isNotNull();
assertThat(restored.getCksumtype()).isEqualTo(mcs.getCksumtype());
assertThat(mcs.getChecksum()).isEqualTo(restored.getChecksum());
- assertThat(restored.tagNo()).isEqualTo(mcs.tagNo());
- assertThat(restored.tagFlags()).isEqualTo(mcs.tagFlags());
}
}
http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/e69ab4d5/kerby-kerb/kerb-core/src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java
----------------------------------------------------------------------
diff --git a/kerby-kerb/kerb-core/src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java b/kerby-kerb/kerb-core/src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java
index 4827961..fd3d4ba 100644
--- a/kerby-kerb/kerb-core/src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java
+++ b/kerby-kerb/kerb-core/src/main/java/org/apache/kerby/kerberos/kerb/KrbCodec.java
@@ -67,7 +67,6 @@ public class KrbCodec {
LimitedByteBuffer limitedBuffer = new LimitedByteBuffer(byteBuffer);
int tag = AbstractAsn1Type.readTag(limitedBuffer);
int tagNo = AbstractAsn1Type.readTagNo(limitedBuffer, tag);
- int tagFlags = tag & 0xe0;
int length = AbstractAsn1Type.readLength(limitedBuffer);
LimitedByteBuffer valueBuffer = new LimitedByteBuffer(limitedBuffer, length);
@@ -90,7 +89,7 @@ public class KrbCodec {
} else {
throw new IOException("To be supported krb message type with tag: " + tag);
}
- msg.decode(tagFlags, tagNo, valueBuffer);
+ msg.decode(tag, tagNo, valueBuffer);
return msg;
}