Cleaning pc

JBSnorro/Collections/Bits/IFloatingPointBitReaderExtensions.cs

@@ -22,6 +22,7 @@ public static double ReadDouble(this IBitReader reader, int bitCount, IFloatingP
     {
         if (reader == null) throw new ArgumentNullException(nameof(reader));
         if (bitCount < IFloatingPointBitReader.MIN_BIT_COUNT) throw new ArgumentOutOfRangeException(nameof(bitCount));
+        if ((ulong)bitCount > reader.RemainingLength) throw new ArgumentOutOfRangeException(nameof(bitCount));
         if (!Enum.IsDefined(floatingPointEncoding)) throw new ArgumentOutOfRangeException(nameof(floatingPointEncoding));
 
         switch (floatingPointEncoding)

JBSnorro/Collections/Bits/Internals/BitReaderWithAlongTagger.cs

@@ -1,4 +1,6 @@
-namespace JBSnorro.Collections.Bits.Internals;
+using JBSnorro.Diagnostics;
+
+namespace JBSnorro.Collections.Bits.Internals;
 
 /// <summary>
 /// A bit reader that reads floating point numbers assuming they're particularly encoded.
@@ -10,6 +12,19 @@ internal class BitReaderWithAlongTagger : BitReader
     public BitReaderWithAlongTagger(BitArray data, ulong position, ulong length, BitReader alongTagger)
         : base(data, position, length)
     {
+        Contract.Requires(alongTagger is not null);
+#if DEBUG
+        var set = new HashSet<BitReader>();
+        for (var a = alongTagger; a != null; a = (a as BitReaderWithAlongTagger)?.alongTagger)
+        {
+            if (set.Contains(a))
+            {
+                throw new ArgumentException("Infinite loop");
+            }
+            set.Add(a);
+        }
+#endif
+
         this.alongTagger = alongTagger;
     }
 
@@ -21,12 +36,18 @@ protected internal override ulong current
         }
         set
         {
+            if (this.alongTagger is null)
+            {
+                // we're called from the base constructor
+                base.current = value;
+                return;
+            }
             unchecked
             {
                 var delta = value - current;
                 if (delta != 0)
                 {
-                    this.current = value;
+                    base.current = value;
                     alongTagger.current += delta;
                 }
             }

JBSnorro/Collections/Bits/Internals/ULongLikeFloatingPointBitReader.cs

@@ -8,6 +8,8 @@ internal static double ReadDouble(IBitReader reader, int bitCount)
     {
         if (bitCount < IFloatingPointBitReader.MIN_BIT_COUNT || bitCount > 64)
             throw new ArgumentOutOfRangeException(nameof(bitCount));
+        if ((ulong)bitCount > reader.RemainingLength)
+            throw new ArgumentOutOfRangeException(nameof(bitCount));
 
         ulong value = reader.ReadUInt64(bitCount);
         if (value == 0)