]> git.saurik.com Git - redis.git/commit
Make sure that SELECT argument is an integer or return an error.
authorantirez <antirez@gmail.com>
Tue, 11 Sep 2012 08:32:04 +0000 (10:32 +0200)
committerantirez <antirez@gmail.com>
Tue, 11 Sep 2012 08:34:48 +0000 (10:34 +0200)
commitb58f03a0e858bd55f61f8865bfb58bdbb45c232b
tree9f5dc06850dbd544a897222c670cd4297a6a36d4
parentefb54f0593dfa01f7c8a81ad216559e87a85497a
Make sure that SELECT argument is an integer or return an error.

Unfortunately we had still the lame atoi() without any error checking in
place, so "SELECT foo" would work as "SELECT 0". This was not an huge
problem per se but some people expected that DB can be strings and not
just numbers, and without errors you get the feeling that they can be
numbers, but not the behavior.

Now getLongFromObjectOrReply() is used as almost everybody else across
the code, generating an error if the number is not an integer or
overflows the long type.

Thanks to @mipearson for reporting that on Twitter.
src/db.c