Fixed crash on ZREM with missing parameter.

generic_multiple_args_cmd() checked for missing
parameters but was missing a return FAILURE when
the number of parameters was lower than expected.

A unit test was added. This fixes GiHub issue phpredis#102.

Author: nicolasff

redis.c

@@ -1974,7 +1974,7 @@ PHPAPI int generic_multiple_args_cmd(INTERNAL_FUNCTION_PARAMETERS, char *keyword
     RedisSock *redis_sock;
 
     if(argc < min_argc) {
-        WRONG_PARAM_COUNT;
+		zend_wrong_param_count();
 		ZVAL_BOOL(return_value, 0);
 		return FAILURE;
     }

tests/TestRedis.php

@@ -2686,6 +2686,10 @@ private function checkSerializer($mode) {
 	    $this->assertTrue(1 === $this->redis->zDelete('key', $z[3]));
 	    $this->assertTrue(0 === $this->redis->zDelete('key', $z[3]));
 	    unset($z[3]);
+
+		// check that zDelete doesn't crash with a missing parameter (GitHub issue #102):
+		$this->assertTrue(FALSE === @$this->redis->zDelete('key'));
+
 	    // variadic
 	    $this->redis->delete('k');
 	    $this->redis->zAdd('k', 0, 'a');