Merge pull request #20 from ilylia/master

fix superfluous leading zeros
vjeantet · Sep 19, 2017 · 479fece · 479fece
2 parents c3e1da8 + 22e164f
commit 479fece
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/packet.go b/packet.go
@@ -118,11 +118,13 @@ func readTagAndLength(conn *bufio.Reader, bytes *[]byte) (ret ldap.TagAndLength,
 			}
 			ret.Length <<= 8
 			ret.Length |= int(b)
-			if ret.Length == 0 {
-				// DER requires that lengths be minimal.
-				err = ldap.StructuralError{"superfluous leading zeros in length"}
-				return
-			}
+			// Compat some lib which use go-ldap or someone else,
+			// they encode int may have leading zeros when it's greater then 127
+			// if ret.Length == 0 {
+			// 	// DER requires that lengths be minimal.
+			// 	err = ldap.StructuralError{"superfluous leading zeros in length"}
+			// 	return
+			// }
 		}
 	}