fluencelabs/redis
1
0
Fork 0
mirror of https://github.com/fluencelabs/redis synced 2025-03-19 17:10:50 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #3256 from oranagra/georasius_neg

Browse Source 
fix georadius returns multiple replies
This commit is contained in:
Salvatore Sanfilippo 2016-06-16 12:57:59 +02:00 committed by GitHub
commit 64b834b8b2
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/geo.c
View File

@ -156,9 +156,13 @@ double extractDistanceOrReply(client *c, robj **argv,
return -1; return -1;
} }
if (distance < 0) {
addReplyError(c,"radius cannot be negative");
return -1;
}
double to_meters = extractUnitOrReply(c,argv[1]); double to_meters = extractUnitOrReply(c,argv[1]);
if (to_meters < 0) { if (to_meters < 0) {
addReplyError(c,"radius cannot be negative");
return -1; return -1;
} }
@ -465,7 +469,6 @@ void georadiusGeneric(client *c, int type) {
double radius_meters = 0, conversion = 1; double radius_meters = 0, conversion = 1;
if ((radius_meters = extractDistanceOrReply(c, c->argv + base_args - 2, if ((radius_meters = extractDistanceOrReply(c, c->argv + base_args - 2,
&conversion)) < 0) { &conversion)) < 0) {
addReplyError(c,"radius must be >= 0");
return; return;
} }