Skip to content
Snippets Groups Projects
Forked from NebulousLabs / Sia
Source project has a limited visibility.
  • David Vorick's avatar
    b973637f
    fix locking in consensus package (#1093) · b973637f
    David Vorick authored and Luke Champine's avatar Luke Champine committed
    There were a few locking bugs in the consensus package, places where the
    database was being accesed without the readlock.
    
    There was one critical place in particular, which was using a readlock
    followed by a writelock, when it really needed to be using a single
    writelock -> demotelock. This place is likely the cause of the consensus
    desynchronizations between the miner + consensus package, has likely
    impacted other modules, and since fixing it I have not been able to
    trigger the behavior in question.
    b973637f
    History
    fix locking in consensus package (#1093)
    David Vorick authored and Luke Champine's avatar Luke Champine committed
    There were a few locking bugs in the consensus package, places where the
    database was being accesed without the readlock.
    
    There was one critical place in particular, which was using a readlock
    followed by a writelock, when it really needed to be using a single
    writelock -> demotelock. This place is likely the cause of the consensus
    desynchronizations between the miner + consensus package, has likely
    impacted other modules, and since fixing it I have not been able to
    trigger the behavior in question.