- 27 Aug, 2014 40 commits
-
-
Matt Stancliff authored
Clearly ip[32] is wrong, but it's less clear that buf[32] was wrong without further reading.
-
Eiichi Sato authored
Closes #1914
-
Jan-Erik Rediger authored
Once this is merged: - merge the latest changes to commands.json in antirez/redis-doc - re-run: utils/generate-command-help.rb > src/help.h Then we'll have nice and easy tab-completed help in redis-cli again. Closes #1909
-
Mariano Pérez Rodríguez authored
Uses ANSI "default background" color code after closing tests so any non-black terminals don't remain polluted. Fixes #1649 Closes #1912
-
Matt Stancliff authored
Sentinel needs to die with a more accurate error message when attempted to open a config from STDIN. See: https://groups.google.com/forum/#!topic/redis-db/sYx7VNMWaNM
-
Matt Stancliff authored
It has an important name, but nothing uses it...
-
antirez authored
-
Cong Ding authored
Closes #878
-
David Palm authored
Also adds a fallthrough case for when given large values (like overflow numbers of 2^64 by mistake). Closes #858
-
yoav authored
Closes #857
-
Dov Murik authored
Fix redis-cli prompt to state "not connected" after a SHUTDOWN command is sent.
-
xiaoyu authored
d is more clear because the type of argument is dict not dictht Closes #513
-
antirez authored
The bug was found by @CAFxX, thanks! See issue #1940.
-
antirez authored
-
antirez authored
-
antirez authored
This raises the max string to 4GB without any downside.
-
Pieter Noordhuis authored
Hat tip to @rfuchs. See: https://github.com/redis/hiredis/pull/178. Fixes #1187
-
charsyam authored
Also moves acceptHandler() to be near the other accept...() functions. Closes #1105
-
NanXiao authored
For some Solaris flavours, the inet_aton in in resolv library. Not linking this library will introduce link error. Improves compatability with older Solaris and still works on new Solaris. Closes #1092
-
Daniel Price authored
Closes #997
-
Matt Stancliff authored
Closes #1161
-
Chris Lamb authored
Found by The Mayhem Team (Alexandre Rebert, Thanassis Avgerinos, Sang Kil Cha, David Brumley, Manuel Egele) Cylab, Carnegie Mellon University. See http://bugs.debian.org/716259 for more. Signed-off-by:
Chris Lamb <lamby@debian.org> Fixes #1191
-
charsyam authored
Closes #1129
-
Matt Stancliff authored
Previously, "MOVE key somestring" would move the key to DB 0 which is just unexpected and wrong. String as DB == error. Test added too. Modified by @antirez in order to use the getLongLongFromObject() API instead of strtol(). Fixes #1428
-
Matt Stancliff authored
Also adds test for numsub — due to tcl being tcl, it doesn't capture the "numberness" of the fix, but now we at least have one test case for numsub. Closes #1561
-
Xiaojie Zhang authored
Closes #1519
-
Samuel Chou authored
[I had to split out the clang check due to clang *really* not liking the __GLIBC_PREREQ macro; -matt] Closes #1456
-
Jim Howard authored
Reason: https://github.com/antirez/redis/pull/1450#issuecomment-30432925 Closes #1450
-
Xian Li authored
Closes #1614
-
Joshua Anderson authored
Closes #1610
-
cubicdaiya authored
According to the C standard, it is desirable to give the type 'void' to functions have no argument. Closes #1631
-
Matt Stancliff authored
dictAdd returns DICT_OK, not REDIS_OK. They both have the same underlying values, so it works even though the code is technically wrong. Fixes #1512
-
Jan-Erik Rediger authored
This was discovered by _bodya and reported in the IRC channel. Everything worked fine as these scripts are always executed as shell scripts. Closes #1728
-
Edgars Irmejs authored
According to unix manuals, "Connecting to the socket object requires read/write permission." -- mode 755 is useless for anybody other than the owner. Fixes #1696
-
ripcurld00d authored
Use constants to avoid magic numbers in `types`, which is an array that stores the names of `REDIS` types. Closes #1681
-
Kashif Rasul authored
Modified by @antirez since the original fix to genInfoString() looked weak. Probably the clang analyzer complained about `section` being possibly NULL, and strcasecmp() called with a NULL pointer. In the practice this can never happen, still for the sake of correctness the right fix is not to modify only the first call, but to set `section` to the value of "default" if it happens to be NULL. Closes #1660
-
antirez authored
Also quit ASAP when we are still loading a DB, since care is not needed in this special condition, especially for a SIGINT.
-
Matt Stancliff authored
-
Matt Stancliff authored
The classic (min+max)/2 is provably unsafe. Fixed as recommended in research: http://googleresearch.blogspot.com/2006/06/extra-extra-read-all-about-it-nearly.html Fix inspired by @wjin, but I used a different approach. (later, I found @kuebler fixed the same issue too). Fixes #1741, #1602
-
Jan-Erik Rediger authored
Previously the end was casted to a smaller type which resulted in a wrong check and failed with values larger than handled by unsigned. Closes #1847, #1844
-