293348d0de
Missing va_end also added in hiredis copy of sds.c.
2014-08-26 10:05:32 +02:00
26744da0d1
Merge pull request #1964 from mariano-perez-rodriguez/patch-2
...
Missing assert removal from sdsIncrLen()
2014-08-26 10:03:54 +02:00
8bbc1897c4
Missing assert removal from sdsIncrLen()
...
Companion for 8eeb1802ec
2014-08-25 15:24:15 -03:00
119a7000b5
Use modern typedef form in cluster.h.
2014-08-25 10:42:18 +02:00
8eeb1802ec
No longer useful assert removed from sdsIncrLen().
2014-08-25 10:36:26 +02:00
209f266cc5
Merge branch '1906-merge' into unstable
2014-08-25 10:27:53 +02:00
1f8a6d806c
Fix redis-benchmark abort output
2014-08-25 10:26:30 +02:00
6b099c7de6
Deny CLIENT command in scripts
...
We don't want scripts doing CLIENT SETNAME
or CLIENT KILL or CLIENT LIST or CLIENT PAUSE.
Originally reported by Chris Wj then proper
action inspired by Itamar Haber.
Reference: https://groups.google.com/forum/#!topic/redis-db/09B2EYwyVgk
2014-08-25 10:25:30 +02:00
aa404fc11b
redis-benchmark: Remove unused buffer
...
This just deletes old code that didn't get removed when
logic changed. We were setting offsets that never
got read anywhere.
Since clients are now just cloned, we don't need to track
per-client buffer offsets anywhere because they are all
the same from the original client.
2014-08-25 10:19:43 +02:00
b34a8c3b75
redis-benchmark: add auth option
...
Closes #1097
2014-08-25 10:17:33 +02:00
7e63dd23f3
Rename two 'buf' vars to 'ip' for better clarity
...
Clearly ip[32] is wrong, but it's less clear that buf[32] was wrong
without further reading.
2014-08-25 10:16:20 +02:00
c38884ceac
Sentinel: fix bufsize to support IPv6 address
...
Closes #1914
2014-08-25 10:15:43 +02:00
ef57f94df0
Use correct github url to find commands.json
...
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
2014-08-25 10:15:28 +02:00
5afe1e37c7
Stop tests from leaving a black background
...
Uses ANSI "default background" color code after closing tests
so any non-black terminals don't remain polluted.
Fixes #1649
Closes #1912
2014-08-25 10:14:03 +02:00
681de88df4
Sentinel: Reject config from STDIN
...
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
2014-08-25 10:13:34 +02:00
60c448b584
Cluster: Fix segfault if cluster config corrupt
...
This commit adds a size check after initial config
line parsing to make sure we have *at least* 8 arguments
per line.
Also, instead of asserting for cluster->myself, we just test
and error out normally (since the error does a hard exit anyway).
Closes #1597
2014-08-25 10:11:38 +02:00
879e18b7ec
Fix memory leak in cluster config parsing
...
The continue stop us from triggering the
free after the long line for loop, so add it
earlier.
2014-08-18 11:27:19 +02:00
82bac1b729
Remove unused global variable
...
It has an important name, but nothing uses it...
2014-08-18 11:23:58 +02:00
a6edfceaa8
Fix invalid expire error for SET family commands.
2014-08-18 11:15:50 +02:00
18ca831830
Remove unused function
...
Closes #878
2014-08-18 11:12:26 +02:00
100c3315be
Extend range of bytesToHuman to TB and PB
...
Also adds a fallthrough case for when given
large values (like overflow numbers of 2^64 by mistake).
Closes #858
2014-08-18 11:11:00 +02:00
0a98b21f65
Add error check for writing RDB checksum
...
Closes #857
2014-08-18 11:09:06 +02:00
233d24a737
redis-cli: fix prompt after shutdown command
...
Fix redis-cli prompt to state "not connected" after a SHUTDOWN command
is sent.
2014-08-18 11:05:36 +02:00
d786fb6e94
Clarify argument to dict macro
...
d is more clear because the type of argument is dict not dictht
Closes #513
2014-08-18 10:59:01 +02:00
6a7a32a806
Clarify existing slot wording on cluster start
2014-08-18 10:58:00 +02:00
c951c3ee5a
Fix undefined behavior in ll2string().
...
The bug was found by @CAFxX, thanks!
See issue #1940 .
2014-08-15 15:48:15 +02:00
367035a983
Don't use extended Regexp Syntax
...
It's not POSIX (BSD systems have -E instead) and we don't actually need it.
Closes #1922
2014-08-13 19:34:03 +02:00
edca2b14d2
Remove warnings and improve integer sign correctness.
2014-08-13 11:44:38 +02:00
7e3ccddc6d
Add -W to compilation flags.
2014-08-13 10:53:54 +02:00
68db7b1f56
Use unsigned integers in SDS header.
...
This raises the max string to 4GB without any downside.
2014-08-13 10:53:51 +02:00
cf85b5ba81
va_copy must be matched by va_end
...
Hat tip to @rfuchs. See: https://github.com/redis/hiredis/pull/178 .
Fixes #1187
2014-08-12 11:50:02 +02:00
3426866e0a
Remove duplicate prototypes in redis.h
...
Also moves acceptHandler() to be near the other accept...() functions.
Closes #1105
2014-08-12 11:45:25 +02:00
951eefae3c
Use resolv library in Solaris
...
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
2014-08-12 11:30:46 +02:00
b57a4d07a2
Define AR to help with lua cross-compilation
...
Closes #997
2014-08-12 11:26:34 +02:00
ded57795ff
representRedisNodeFlags() moved into right code section.
...
The funciton was also modified in order to be more standalone and
produce an output without trailing spaces, making the reuse simpler.
The global variable was renamed in cammel case as most other Redis
globals, except the main ones we refer too many times, like 'server'.
2014-08-08 15:53:42 +02:00
de5465baf7
Refactor cluster flag printing
...
Less copy/paste code duplication.
Closes #952
2014-08-08 15:39:44 +02:00
da0132638f
Cleanup double semicolons
...
Closes #1161
2014-08-08 14:54:02 +02:00
dec58464d8
Free memory in clusterLoadConfig error handler
...
Closes #1327
2014-08-08 14:40:32 +02:00
af62d9a687
redis-check-dump: Prevent segfault if can't malloc
...
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
2014-08-08 14:10:34 +02:00
354119e4c5
Remove unused LINE_BUFLEN definition
...
Closes #1129
2014-08-08 14:05:02 +02:00
498ad7482b
Reject MOVE to non-integer DBs
...
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
2014-08-08 12:38:43 +02:00
25791550e0
pubsub: Return integers for NUMSUB, not strings
...
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
2014-08-08 11:19:37 +02:00
4bb6844e43
Avoid unnecessary decoding in ziplist.c
...
Closes #1519
2014-08-08 11:04:54 +02:00
7e9f24d694
Improve accuracy of HAVE_ATOMIC dependency check
...
[I had to split out the clang check due to
clang *really* not liking the __GLIBC_PREREQ macro; -matt]
Closes #1456
2014-08-08 10:46:57 +02:00
a8d3e930bc
Remove redundant event loop fd processing
...
Reason:
https://github.com/antirez/redis/pull/1450#issuecomment-30432925
Closes #1450
2014-08-08 10:43:54 +02:00
f510cfb4ef
memtest: Add missing free()
...
Closes #1614
2014-08-08 10:30:14 +02:00
7a2450a7e5
src/Makefile: Reword "to run make test" message
...
Closes #1610
2014-08-08 10:20:22 +02:00
23f08510d5
Use 'void' for zero-argument functions
...
According to the C standard,
it is desirable to give the type 'void'
to functions have no argument.
Closes #1631
2014-08-08 10:05:32 +02:00
8db020e2a1
Fix assert technical correctness
...
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
2014-08-08 10:03:22 +02:00
6df1251baa
install_server.sh: add missing bang
...
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
2014-08-07 17:14:08 +02:00
