der: add IsConstructed trait, impl'ed on any FixedTag

der_derive: impl IsConstructed on derive(Choice)

der: draft of test for IsConstructed vs Tagged edge case

der: test: generic CHOICE inside [0] IMPLICIT CHOICE

Revert "der: test: generic CHOICE inside [0] IMPLICIT CHOICE"

This reverts commit dda1215.

Revert "der: draft of test for IsConstructed vs Tagged edge case"

This reverts commit 5daa9ba.

der: add IsConstructed test

docs

Author: dishmaker

der/src/asn1/context_specific.rs

@@ -3,6 +3,7 @@
 use crate::{
     Choice, Decode, DecodeValue, DerOrd, Encode, EncodeValue, EncodeValueRef, Error, Header,
     Length, Reader, Tag, TagMode, TagNumber, Tagged, ValueOrd, Writer, asn1::AnyRef,
+    tag::IsConstructed,
 };
 use core::cmp::Ordering;
 
@@ -60,13 +61,13 @@ impl<T> ContextSpecific<T> {
         tag_number: TagNumber,
     ) -> Result<Option<Self>, T::Error>
     where
-        T: DecodeValue<'a> + Tagged,
+        T: DecodeValue<'a> + IsConstructed,
     {
         Self::decode_with::<_, _, T::Error>(reader, tag_number, |reader| {
             let header = Header::decode(reader)?;
             let value = T::decode_value(reader, header)?;
 
-            if header.tag.is_constructed() != value.tag().is_constructed() {
+            if header.tag.is_constructed() != T::CONSTRUCTED {
                 return Err(header.tag.non_canonical_error().into());
             }
 

der/src/lib.rs

@@ -377,7 +377,7 @@ pub use crate::{
     length::{IndefiniteLength, Length},
     ord::{DerOrd, ValueOrd},
     reader::{Reader, slice::SliceReader},
-    tag::{Class, FixedTag, Tag, TagMode, TagNumber, Tagged},
+    tag::{Class, FixedTag, IsConstructed, Tag, TagMode, TagNumber, Tagged},
     writer::{Writer, slice::SliceWriter},
 };
 

der/src/tag.rs

@@ -32,6 +32,17 @@ impl<T: FixedTag + ?Sized> Tagged for T {
     }
 }
 
+/// Types which have a constant ASN.1 constructed bit.
+pub trait IsConstructed {
+    /// ASN.1 constructed bit
+    const CONSTRUCTED: bool;
+}
+
+/// Types which are [`FixedTag`] always known if they are constructed (or primitive).
+impl<T: FixedTag + ?Sized> IsConstructed for T {
+    const CONSTRUCTED: bool = T::TAG.is_constructed();
+}
+
 /// ASN.1 tags.
 ///
 /// Tags are the leading identifier octet of the Tag-Length-Value encoding
@@ -229,7 +240,7 @@ impl Tag {
     }
 
     /// Does this tag represent a constructed (as opposed to primitive) field?
-    pub fn is_constructed(self) -> bool {
+    pub const fn is_constructed(self) -> bool {
         match self {
             Tag::Sequence | Tag::Set => true,
             Tag::Application { constructed, .. }

der/tests/derive.rs

@@ -706,6 +706,69 @@ mod decode_value {
     }
 }
 
+/// Custom derive test cases for the `DecodeValue` + `EncodeValue` macro combo.
+mod decode_encode_value {
+    use der::{
+        Decode, DecodeValue, Encode, EncodeValue, FixedTag, Header, IsConstructed, Length,
+        Sequence, SliceReader, SliceWriter, Tag, TagNumber,
+    };
+    use hex_literal::hex;
+
+    /// Example of a structure, that does not have a tag
+    #[derive(DecodeValue, EncodeValue, Default, Eq, PartialEq, Debug)]
+    #[asn1(tag_mode = "IMPLICIT")]
+    struct DecodeEncodeCheck<'a> {
+        #[asn1(type = "OCTET STRING", context_specific = "5")]
+        field: &'a [u8],
+    }
+    impl IsConstructed for DecodeEncodeCheck<'_> {
+        const CONSTRUCTED: bool = true;
+    }
+
+    // TODO(dishmaker): fix test after IMPLICIT/EXPLICIT trait split
+    // #[derive(Sequence, Default, Eq, PartialEq, Debug)]
+    // #[asn1(tag_mode = "IMPLICIT")]
+    // struct ImplicitWrapper<'a> {
+    //     #[asn1(context_specific = "0")]
+    //     implicit_decode_encode: DecodeEncodeCheck<'a>,
+    // }
+
+    #[test]
+    fn sequence_decode_encode_custom_implicit() {
+        // TODO(dishmaker): fix test after IMPLICIT/EXPLICIT trait split
+        // let obj = ImplicitWrapper {
+        //     implicit_decode_encode: DecodeEncodeCheck {
+        //         field: &[0x33, 0x44],
+        //     },
+        // };
+
+        // let der_encoded = obj.to_der().unwrap();
+
+        // assert_eq!(der_encoded, hex!("80 04 85 02 33 44"));
+        // let obj_decoded = ImplicitWrapper::from_der(&der_encoded);
+        // assert_eq!(obj, obj_decoded);
+
+        let header = Header {
+            tag: Tag::ContextSpecific {
+                constructed: true,
+                number: TagNumber(0),
+            },
+            length: Length::new(6u16),
+        };
+        let obj = DecodeEncodeCheck {
+            field: &[0x33, 0x44],
+        };
+
+        let mut buf = [0u8; 100];
+        let mut writer = SliceWriter::new(&mut buf);
+        obj.encode_value(&mut writer).unwrap();
+        let len = obj.value_len().unwrap();
+
+        let mut reader = SliceReader::new(&mut buf).unwrap();
+        DecodeEncodeCheck::decode_value(&mut reader, header);
+    }
+}
+
 /// Custom derive test cases for the `BitString` macro.
 #[cfg(feature = "std")]
 mod bitstring {

der_derive/src/choice.rs

@@ -97,6 +97,10 @@ impl DeriveChoice {
                 }
             }
 
+            impl #impl_generics ::der::IsConstructed for #ident #ty_generics #where_clause {
+                const CONSTRUCTED: bool = true;
+            }
+
             impl #impl_generics ::der::Decode<#lifetime> for #ident #ty_generics #where_clause {
                 type Error = #error;