Skip to content

Sanitizer logic is wrong

Some code as the sanitizer use the selectSubnet(address) method but in fact call selectSubnet(address, empty-class-list) so a guarded subnet never matches. I propose:

  • first to make the second argument not optional so we never get bad calls to selectSubnet.
  • second to change the logic of the sanitizer to check first if the lease subnet-id is valid (subnet exists and the address is in range) before looking for a candidate on mismatch.
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information