Bugfixes in redis_session.c

Author: nicolasff

redis_session.c

@@ -106,6 +106,7 @@ redis_pool_get_sock(redis_pool *pool, const char *key TSRMLS_DC) {
 			return rpm->redis_sock;
 		}
 		i += rpm->weight;
+        rpm = rpm->next;
 	}
 
 	return NULL;
@@ -133,7 +134,8 @@ PS_OPEN_FUNC(redis)
 			 j++;
 
 		if (i < j) {
-			int weight = 1, timeout = 86400;
+			int weight = 1;
+			double timeout = 86400.0;
 
 			/* unix: isn't supported yet. */
 			if (!strncmp(save_path+i, "unix:", sizeof("unix:")-1)) {
@@ -173,8 +175,7 @@ PS_OPEN_FUNC(redis)
 				}
 
 				if (zend_hash_find(Z_ARRVAL_P(params), "timeout", sizeof("timeout"), (void **) &param) != FAILURE) {
-					convert_to_long_ex(param);
-					timeout = Z_LVAL_PP(param);
+					timeout = atof(Z_STRVAL_PP(param));
 				}
 
 				/* // not supported yet
@@ -287,7 +288,7 @@ PS_WRITE_FUNC(redis)
 
 	/* send SET command */
 	session = redis_session_key(key, strlen(key), &session_len);
-	cmd_len = redis_cmd_format_static(&cmd, "SET", "ss", session, session_len, val, vallen);
+	cmd_len = redis_cmd_format_static(&cmd, "SETEX", "sds", session, session_len, INI_INT("session.gc_maxlifetime"), val, vallen);
 	efree(session);
 	if(redis_sock_write(redis_sock, cmd, cmd_len) < 0) {
 		efree(cmd);