Enables AMF to accept not-perfect NGAP packets which have not 0 ASN.1…

… PER padding bit. (#2) * Enables AMF to accept not-perfect NGAP packets which have not 0 ASN.1 PER padding bit. * Use go build -tags skip_padding_check to switch padding check Default will check padding if value is 0 Co-authored-by: free5gc-org <[email protected]>
free5gc · Apr 1, 2022 · 504c704 · 504c704
1 parent 162f622
commit 504c704
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/aper.go b/aper.go
@@ -131,7 +131,12 @@ func (pd *perBitData) parseAlignBits() error {
 		if val, err := pd.getBitsValue(alignBits); err != nil {
 			return err
 		} else if val != 0 {
-			return fmt.Errorf("Align Bit is not zero")
+			if skipPaddingCheck {
+				perTrace(2, "Align Bit is not zero")
+				perTrace(1, perBitLog(uint64(alignBits), pd.byteOffset, pd.bitsOffset, val))
+			} else {
+				return fmt.Errorf("Align Bit is not zero")
+			}
 		}
 	} else if pd.bitsOffset != 0 {
 		pd.bitCarry()

diff --git a/padding_check.go b/padding_check.go
@@ -0,0 +1,5 @@
+// +build !skip_padding_check
+
+package aper
+
+const skipPaddingCheck bool = false
diff --git a/skip_padding_check.go b/skip_padding_check.go
@@ -0,0 +1,5 @@
+// +build skip_padding_check
+
+package aper
+
+const skipPaddingCheck bool = true