CoreyKaylor / Lightning.NET

.NET library for LMDB key-value store
Other
397 stars 82 forks source link

ArithmeticOverflow in EnvironmentInfo #135

Closed pepelev closed 3 years ago

pepelev commented 3 years ago

Hi!

There is arithmetic overflow in getting EnvironmentInfo. One can checkout penultimate commit to get test failed. This is reproducible on Windows.

AlgorithmsAreCool commented 3 years ago

Thank you very much!

I'm amazed we didn't notice the MapSize limitation before?!

I'll merge this after a few hours to allow time for any other comments, but LGTM.