diff options
author | Matt Stancliff <matt@genges.com> | 2015-01-05 12:13:32 -0500 |
---|---|---|
committer | Matt Stancliff <matt@genges.com> | 2015-01-05 16:52:08 -0500 |
commit | 6a00a4643baaa920b182eb65ee82d9b8c23e3134 (patch) | |
tree | 89a12550994dbeccb29cc98f2acafda5dbe3055c /COPYING | |
parent | 0c9ff5bb030fd07706c39b3d4f28cef618dd8794 (diff) |
Fix redisAppendCommand error result
Previously, redisvAppendCommand() would return OOM even with formatting
errors. Now we use OTHER with an error string telling the user the
error was formatting related, not memory related.
This also fixes a potentially worse bug where were would pass error result
of -1 as an actual length to another function taking an unsigned length,
which would result in crash/overallocation/errors. Now for that case,
we just return an error immediately and stop processing the command.
Fixes #177
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions