search

Netflix / dynomite

A generic dynamo implementation for different k-v storage engines
Apache License 2.0
4.2k stars 534 forks source link

Fix conf parsing overflow #664

Closed smukil closed 5 years ago

smukil commented 5 years ago

After adding a new option to the 'struct conf_pool' list, a previously undiscovered bug came to light where we were casting data types of 'short' to 'int' and 'bool' to 'int' which would give us the wrong result. This patch fixes these bugs.

smukil commented 5 years ago

@shailesh33 Thanks for the review!