search

yannh / redis-dump-go

Backup & Restore your Redis server - FAST
MIT License
274 stars 58 forks source link

fix: nil pointer dereference when selecting database #34

Closed sonirico closed 2 years ago

sonirico commented 2 years ago

Prior to this commit the fact of selecting which redis database to dump when making use of the option -db {redis_db} would yield a nil pointer de-reference error due to the fact that a nil pointer can not be de-referenced even if it is an assignment expression.

yannh commented 2 years ago

Urgh. Thanks, I'll also add an acceptance test for this :100:

sonirico commented 2 years ago

@yannh Ok, just tell me if you need help of any kind. It'd be great to have this in upstream :) Tagged v0.6.1

yannh commented 2 years ago

Merged with some new tests, thanks a lot for the fix :)