hiredis 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d434e6b09126833f2a6dcf82385105de08f629dd
-  data.tar.gz: 2d668be20b90a469f8f905f5915868e143cf5ae4
+  metadata.gz: 96f77b981784a483844d0f71fb48594a0a79fced
+  data.tar.gz: e6c6f088e0feb373de9097838de379870413791b
 SHA512:
-  metadata.gz: 637b4bc535ceca4c61aff1f3a1d7456819dc9abdfc934618eac3f8c66b2d791a8106197c552ee6a8635611bcb8588cebb4d183a0de60e21ad4b8b10a29f494eb
-  data.tar.gz: 5170e3ae0d95f728b2ca8566717de256aba4a4e3f804618a968ce81bbe8e3fca5b57942fe3d050441c10fe44aaf3f3c140ed7d8c99bc45e153033cb34884ad99
+  metadata.gz: 7bc1f73b803eab73504898440c7cbe001abacad798e94b21d683178ec7d6b12a6104b5a5a6f53949dc571625a5a8ccca3576c04a700478d25a3da0e82577e546
+  data.tar.gz: e89b77e064f841199206d5ced3f272120ac59e2234d79e947fdfe25fb61370407b215682e857785ae3b46681d3398f37bf54a2a7afe24e0218e6010f44aaed94

data/ext/hiredis_ext/connection.c

@@ -110,16 +110,20 @@ static int __wait_readable(int fd, const struct timeval *timeout, int *isset) {
     struct timeval *toptr = NULL;
 
     _fdset_t fds;
+
+    /* Be cautious: a call to rb_fd_init to initialize the rb_fdset_t structure
+     * must be paired with a call to rb_fd_term to free it. */
     _fd_init(&fds);
     _fd_set(fd, &fds);
 
-    /* rb_fd_select modifies the passed timeval, so we pass a copy */
+    /* rb_thread_{fd_,}select modifies the passed timeval, so we pass a copy */
     if (timeout != NULL) {
         memcpy(&to, timeout, sizeof(to));
         toptr = &to;
     }
 
     if (_thread_fd_select(fd + 1, &fds, NULL, NULL, toptr) < 0) {
+        _fd_term(&fds);
         return -1;
     }
 
@@ -127,6 +131,7 @@ static int __wait_readable(int fd, const struct timeval *timeout, int *isset) {
         *isset = 1;
     }
 
+    _fd_term(&fds);
     return 0;
 }
 
@@ -135,16 +140,20 @@ static int __wait_writable(int fd, const struct timeval *timeout, int *isset) {
     struct timeval *toptr = NULL;
 
     _fdset_t fds;
+
+    /* Be cautious: a call to rb_fd_init to initialize the rb_fdset_t structure
+     * must be paired with a call to rb_fd_term to free it. */
     _fd_init(&fds);
     _fd_set(fd, &fds);
 
-    /* rb_fd_select modifies the passed timeval, so we pass a copy */
+    /* rb_thread_{fd_,}select modifies the passed timeval, so we pass a copy */
     if (timeout != NULL) {
         memcpy(&to, timeout, sizeof(to));
         toptr = &to;
     }
 
     if (_thread_fd_select(fd + 1, NULL, &fds, NULL, toptr) < 0) {
+        _fd_term(&fds);
         return -1;
     }
 
@@ -152,6 +161,7 @@ static int __wait_writable(int fd, const struct timeval *timeout, int *isset) {
         *isset = 1;
     }
 
+    _fd_term(&fds);
     return 0;
 }
 

data/lib/hiredis/version.rb

@@ -1,3 +1,3 @@
 module Hiredis
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hiredis
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Pieter Noordhuis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-27 00:00:00.000000000 Z
+date: 2014-04-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake