Skip to content

Commit 7d44ecc

Browse files
committed
Update IKEv2 tests with new X509_AlgorithmIdentifier
1 parent 1719971 commit 7d44ecc

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

scapy/asn1fields.py

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,7 @@ def build(self, pkt):
599599
elif val is None:
600600
s = b""
601601
else:
602-
s = b"".join(raw(i) for i in val)
602+
s = b"".join(bytes(i) for i in val)
603603
return self.i2m(pkt, s)
604604

605605
def i2repr(self, pkt, x):
@@ -769,7 +769,7 @@ def i2m(self, pkt, x):
769769
if x is None:
770770
s = b""
771771
else:
772-
s = raw(x)
772+
s = bytes(x)
773773
if hash(type(x)) in self.pktchoices:
774774
imp, exp = self.pktchoices[hash(type(x))]
775775
s = BER_tagging_enc(s,
@@ -852,11 +852,12 @@ def i2m(self,
852852
s = x
853853
elif isinstance(x, ASN1_Object):
854854
if x.val:
855-
s = raw(x.val)
855+
print(repr(x), repr(x.val), self.name)
856+
s = bytes(x.val)
856857
else:
857858
s = b""
858859
else:
859-
s = raw(x)
860+
s = bytes(x)
860861
if not hasattr(x, "ASN1_root"):
861862
# A normal Packet (!= ASN1)
862863
return s
@@ -897,7 +898,7 @@ def __init__(self,
897898
self.cls = cls
898899
super(ASN1F_BIT_STRING_ENCAPS, self).__init__( # type: ignore
899900
name,
900-
default and raw(default),
901+
default and bytes(default),
901902
context=context,
902903
implicit_tag=implicit_tag,
903904
explicit_tag=explicit_tag

scapy/layers/x509.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -893,8 +893,10 @@ class X509_AlgorithmIdentifier(ASN1_Packet):
893893
),
894894
lambda pkt: pkt.algorithm.val == "1.2.840.113549.1.3.1",
895895
),
896+
896897
],
897-
ASN1F_optional(ASN1F_NULL("parameters", None)),
898+
# RFC4055 (=1.2.840.113549.1.1.11) / Default
899+
ASN1F_NULL("parameters", 0),
898900
)
899901
)
900902

test/contrib/ikev2.uts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -781,7 +781,7 @@ frames = [
781781
subjectPublicKeyInfo=X509_SubjectPublicKeyInfo(
782782
signatureAlgorithm=X509_AlgorithmIdentifier(
783783
algorithm=ASN1_OID('ecPublicKey'),
784-
parameters=ASN1_OID('prime256v1')),
784+
parameters=ECParameters(curve=ASN1_OID('prime256v1'))),
785785
subjectPublicKey=ECDSAPublicKey(
786786
ecPoint=ASN1_BIT_STRING(
787787
'000001001011011101000101011100101010000110110101110111010001110'
@@ -1125,7 +1125,7 @@ frames = [
11251125
subjectPublicKeyInfo=X509_SubjectPublicKeyInfo(
11261126
signatureAlgorithm=X509_AlgorithmIdentifier(
11271127
algorithm=ASN1_OID('ecPublicKey'),
1128-
parameters=ASN1_OID('prime256v1')
1128+
parameters=ECParameters(curve=ASN1_OID('prime256v1')),
11291129
),
11301130
subjectPublicKey=ECDSAPublicKey(
11311131
ecPoint=ASN1_BIT_STRING(

0 commit comments

Comments
 (0)