diff options
| author | Yang Bodong <bodong.ybd@alibaba-inc.com> | 2021-04-07 15:01:28 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-07 10:01:28 +0300 |
| commit | 4c14e8668cdfa6a967fd3be6e0a159d1156b7ef6 (patch) | |
| tree | 3e91ff494a9b44a920ee281ee229c1365a3213aa /src/object.c | |
| parent | 4de1031d8a4d573b8930cde46a0d5c6ce4799f95 (diff) | |
| download | redis-4c14e8668cdfa6a967fd3be6e0a159d1156b7ef6.tar.gz | |
Fix out of range confusing error messages (XAUTOCLAIM, RPOP count) (#8746)
Fix out of range error messages to be clearer (avoid mentioning 9223372036854775807)
* Fix XAUTOCLAIM COUNT option confusing error msg
* Fix other RPOP and alike error message to mention positive
Diffstat (limited to 'src/object.c')
| -rw-r--r-- | src/object.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/object.c b/src/object.c index ce2f3dc14..b75e547b9 100644 --- a/src/object.c +++ b/src/object.c @@ -727,7 +727,11 @@ int getRangeLongFromObjectOrReply(client *c, robj *o, long min, long max, long * } int getPositiveLongFromObjectOrReply(client *c, robj *o, long *target, const char *msg) { - return getRangeLongFromObjectOrReply(c, o, 0, LONG_MAX, target, msg); + if (msg) { + return getRangeLongFromObjectOrReply(c, o, 0, LONG_MAX, target, msg); + } else { + return getRangeLongFromObjectOrReply(c, o, 0, LONG_MAX, target, "value is out of range, must be positive"); + } } int getIntFromObjectOrReply(client *c, robj *o, int *target, const char *msg) { |