celestiaorg / nmt

Namespaced Merkle Tree
Apache License 2.0
112 stars 39 forks source link

feat: verifies input range in the computeRoot to avoid panic #186

Closed staheri14 closed 1 year ago

staheri14 commented 1 year ago

Overview

Closes https://github.com/celestiaorg/nmt/issues/137

Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #186 (eecd425) into master (1262b7f) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
+ Coverage   95.69%   95.70%   +0.01%     
==========================================
  Files           5        5              
  Lines         557      559       +2     
==========================================
+ Hits          533      535       +2     
  Misses         14       14              
  Partials       10       10              
Impacted Files Coverage Δ
nmt.go 98.01% <100.00%> (+0.01%) :arrow_up: