summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
| * Fix invalid expire error for SET family commands.antirez2014-08-181-1/+1
| |
| * Remove unused functionCong Ding2014-08-181-6/+0
| | | | | | | | Closes #878
| * Extend range of bytesToHuman to TB and PBDavid Palm2014-08-181-0/+9
| | | | | | | | | | | | | | Also adds a fallthrough case for when given large values (like overflow numbers of 2^64 by mistake). Closes #858
| * Add error check for writing RDB checksumyoav2014-08-181-1/+1
| | | | | | | | Closes #857
| * redis-cli: fix prompt after shutdown commandDov Murik2014-08-181-1/+4
| | | | | | | | | | Fix redis-cli prompt to state "not connected" after a SHUTDOWN command is sent.
| * Clarify argument to dict macroxiaoyu2014-08-181-1/+1
| | | | | | | | | | | | d is more clear because the type of argument is dict not dictht Closes #513
| * Clarify existing slot wording on cluster startMatt Stancliff2014-08-181-6/+5
| |
| * Remove warnings and improve integer sign correctness.antirez2014-08-1317-48/+66
| |
| * Add -W to compilation flags.antirez2014-08-131-1/+1
| |
| * Use unsigned integers in SDS header.antirez2014-08-132-4/+7
| | | | | | | | This raises the max string to 4GB without any downside.
| * va_copy must be matched by va_endPieter Noordhuis2014-08-121-0/+1
| | | | | | | | | | | | Hat tip to @rfuchs. See: https://github.com/redis/hiredis/pull/178. Fixes #1187
| * Remove duplicate prototypes in redis.hcharsyam2014-08-121-3/+1
| | | | | | | | | | | | Also moves acceptHandler() to be near the other accept...() functions. Closes #1105
| * Use resolv library in SolarisNanXiao2014-08-121-1/+1
| | | | | | | | | | | | | | | | | | | | 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
| * representRedisNodeFlags() moved into right code section.antirez2014-08-081-38/+41
| | | | | | | | | | | | | | 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'.
| * Refactor cluster flag printingcharsyam2014-08-081-20/+36
| | | | | | | | | | | | Less copy/paste code duplication. Closes #952
| * Cleanup double semicolonsMatt Stancliff2014-08-082-2/+2
| | | | | | | | Closes #1161
| * Free memory in clusterLoadConfig error handlerSungBin_Hong2014-08-081-0/+1
| | | | | | | | Closes #1327
| * redis-check-dump: Prevent segfault if can't mallocChris Lamb2014-08-081-0/+1
| | | | | | | | | | | | | | | | | | | | 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
| * Remove unused LINE_BUFLEN definitioncharsyam2014-08-081-1/+0
| | | | | | | | Closes #1129
| * Reject MOVE to non-integer DBsMatt Stancliff2014-08-081-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * pubsub: Return integers for NUMSUB, not stringsMatt Stancliff2014-08-081-1/+1
| | | | | | | | | | | | | | | | 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
| * Avoid unnecessary decoding in ziplist.cXiaojie Zhang2014-08-081-13/+13
| | | | | | | | Closes #1519
| * Improve accuracy of HAVE_ATOMIC dependency checkSamuel Chou2014-08-081-1/+6
| | | | | | | | | | | | | | [I had to split out the clang check due to clang *really* not liking the __GLIBC_PREREQ macro; -matt] Closes #1456
| * Remove redundant event loop fd processingJim Howard2014-08-081-2/+2
| | | | | | | | | | | | | | Reason: https://github.com/antirez/redis/pull/1450#issuecomment-30432925 Closes #1450
| * memtest: Add missing free()Xian Li2014-08-081-0/+1
| | | | | | | | Closes #1614
| * src/Makefile: Reword "to run make test" messageJoshua Anderson2014-08-081-1/+1
| | | | | | | | Closes #1610
| * Use 'void' for zero-argument functionscubicdaiya2014-08-086-20/+20
| | | | | | | | | | | | | | | | According to the C standard, it is desirable to give the type 'void' to functions have no argument. Closes #1631
| * Fix assert technical correctnessMatt Stancliff2014-08-081-1/+1
| | | | | | | | | | | | | | | | 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
| * redis-check-dump: use names instead of magic numsripcurld00d2014-08-071-1/+3
| | | | | | | | | | | | | | Use constants to avoid magic numbers in `types`, which is an array that stores the names of `REDIS` types. Closes #1681
| * Fix issues raised by clang analyzerKashif Rasul2014-08-072-4/+4
| | | | | | | | | | | | | | | | | | | | | | 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
| * Force quit when receiving a second SIGINT.antirez2014-08-072-2/+14
| | | | | | | | | | Also quit ASAP when we are still loading a DB, since care is not needed in this special condition, especially for a SIGINT.
| * Add graceful exit when Ctrl-C is receivedMatt Stancliff2014-08-071-5/+17
| |
| * Fix intset midpoint selectionMatt Stancliff2014-08-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | 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
| * Fix key extraction for SORTMatt Stancliff2014-08-071-2/+3
| | | | | | | | | | | | | | | | | | | | We only want to use the last STORE key, but we have to record we actually found a STORE key so we can increment the final return key count. Test added to prevent further regression. Closes #1883, #1645, #1647
| * Handle large getrange requestsJan-Erik Rediger2014-08-071-1/+1
| | | | | | | | | | | | | | | | 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
| * scripting: no eval with negative key countMatt Stancliff2014-08-071-0/+3
| | | | | | | | | | | | | | Negative key count causes segfault in Lua functions. Fixes #1842 Closes #1843
| * cluster: fix node connection memory leakkingsumos2014-08-071-1/+3
| | | | | | | | | | | | | | | | | | Cluster leaks memory while connecting due to missing freeaddrinfo() (Commit modified by @antirez. The freeaddrinfo() call was misplaced so in case of no address was bound, the memory leak was still there). Closes #1801
| * redis-cli: fix latency result outputJan-Erik Rediger2014-08-071-4/+8
| | | | | | | | | | | | (Cleaned up a little by @mattsta) Closes #1774
| * Remove redundant else/return blockWei Jin2014-08-071-2/+0
| | | | | | | | Fixes #1741
| * Clarify AIX "#undef hz".antirez2014-08-071-1/+2
| |
| * Add support for compiling on AIXsiahl2014-08-074-2/+21
| | | | | | | | Closes #1900
| * redis-cli: Add --no-raw optionMatt Stancliff2014-08-071-0/+3
| | | | | | | | | | | | | | Some people need formatted output even when they have no interactive tty. Fixes #760
| * redis-cli: stop showing incorrectly selected DBMatt Stancliff2014-08-071-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | Previously redis-cli would happily show "-1" or "99999" as valid DB choices. Now, if the SELECT call returned an error, we don't update the DB number in the CLI. Inspired by @anupshendkar in #1313 Fixes #566, #1313
| * redis-cli: Re-attach selected DB after authMatt Stancliff2014-08-071-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | Previously, if you did SELECT then AUTH, redis-cli would show your SELECT'd db even though it didn't happen. Note: running into this situation is a (hopefully) very limited used case of people using multiple DBs and AUTH all at the same time. Fixes antirez#1639
* | Fix undefined behavior in ll2string().antirez2014-08-151-1/+5
|/ | | | | The bug was found by @CAFxX, thanks! See issue #1940.
* Fail SYNC if background save child aborted due to a signal.Yossi Gottlieb2014-07-281-1/+1
|
* Fix spelling of "stand alone" in ascii artMatt Robenolt2014-07-281-1/+2
| | | Also unified the logic to match `genRedisInfoString`
* Cluster: don't migrate to a master that never had slaves.antirez2014-07-251-1/+9
| | | | | | | | Replica migration algorithm modified so that slaves never try to migrate to masters that were never configured to have slaves in the past. We want the algorithm to take care of masters that remained without *working* slaves, but that used to have slaves according to the cluster configuration.
* Fix util.c compilation by including stdint.h.antirez2014-07-231-0/+1
|
* Faster ll2string() implementation.antirez2014-07-231-21/+80
| | | | | | | | | | | | | | | | Based on ideas documented in this blog post: https://www.facebook.com/notes/facebook-engineering/three-optimization-tips-for-c/10151361643253920 The original code was modified to handle signed integers, reformetted to fit inside the Redis code base, and was stress-tested with a program in order to validate the implementation against snprintf(). Redis was measured to be measurably faster from the point of view of clients in real-world operations because of this change, since sometimes number to string conversion is used extensively (for example every time a GET results into an integer encoded object to be returned to the user).
View Lorry Controller Status