posix 0.0.2 → 0.0.4

data/ext/posix/posix.c

@@ -90,6 +90,7 @@ VALUE posix_execve(VALUE self, VALUE _binary, VALUE _argv, VALUE _envp) {
      * the programmer to set ARGV[0] to something
      * reasonable. */
     char** argv = malloc(sizeof(char*)*RARRAY_LEN(_argv) + 1);
+    argv[0] = NULL;
 
     for (i = 0; i < RARRAY_LEN(_argv); i++) {
         argv[i] = StringValuePtr(RARRAY_PTR(_argv)[i]);
@@ -99,14 +100,19 @@ VALUE posix_execve(VALUE self, VALUE _binary, VALUE _argv, VALUE _envp) {
     /* Construct our environment. Note that this totally ignores the precedent
      * set by Process#spawn, Kernel#exec and fiends */
 
-    char **envp = malloc(sizeof(char**)*RHASH(_envp)->ntbl->num_entries + 1);
-
-    keys = rb_hash_keys(_envp);
-    for (i = 0; i < RARRAY_LEN(keys); i++) {
-        akey = RARRAY_PTR(keys)[i];
-        envk = rb_hash_aref(_envp, akey);
-        asprintf(&envp[i], "%s=%s", StringValuePtr(akey), StringValuePtr(envk));
-        envp[i+1] = NULL; /* Ensure that we're null terminated */
+    char **envp;
+    if (RHASH(_envp)->ntbl) {
+        envp = malloc(sizeof(char**)*RHASH(_envp)->ntbl->num_entries + 1);
+        keys = rb_hash_keys(_envp);
+        for (i = 0; i < RARRAY_LEN(keys); i++) {
+            akey = RARRAY_PTR(keys)[i];
+            envk = rb_hash_aref(_envp, akey);
+            asprintf(&envp[i], "%s=%s", StringValuePtr(akey), StringValuePtr(envk));
+            envp[i+1] = NULL; /* Ensure that we're null terminated */
+    }
+    } else {
+        envp = malloc(sizeof(char**));
+        envp[0] = NULL;
     }
 
     execve(binary, argv, envp);

data/lib/posix/sigset.rb

@@ -12,6 +12,7 @@ class Posix
       end
 
       @signals << signal unless include? signal
+      self
     end
 
     def include?(signal)

data/posix.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "posix"
-  s.version     = "0.0.2"
+  s.version     = "0.0.4"
   s.authors     = ["Richo Healey"]
   s.email       = ["richo@psych0tik.net"]
   s.homepage    = "http://github.com/richo/ruby-posix"

data/test/sigprocmask.rb

@@ -6,4 +6,4 @@ mask << "USR2"
 
 puts Posix.sigprocmask(Posix::SIG_SETMASK, mask)
 
-Posix.execve("/bin/bash", ["/bin/bash"], {"rawr" => "Thing"})
+Posix.execve("/bin/bash", ["/bin/bash"], {})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: posix
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors: