Skip to content
  • Sebastian Hahn's avatar
    Sanity-check consensus param values · 026e7987
    Sebastian Hahn authored
    We need to make sure that the worst thing that a weird consensus param
    can do to us is to break our Tor (and only if the other Tors are
    reliably broken in the same way) so that the majority of directory
    authorities can't pull any attacks that are worse than the DoS that
    they can trigger by simply shutting down.
    
    One of these worse things was the cbtnummodes parameter, which could
    lead to heap corruption on some systems if the value was sufficiently
    large.
    
    This commit fixes this particular issue and also introduces sanity
    checking for all consensus parameters.
    026e7987