ruby-ripmime 0.1.1 → 0.1.2

data/ext/ruby-ripmime.c

@@ -10,6 +10,17 @@ static VALUE ruby_ripmime_decode(VALUE self, VALUE mailpack, VALUE outputdir) {
   VALUE errmsg;
   char buf[BUFSIZ];
 
+  Check_Type(mailpack, T_STRING);
+  Check_Type(outputdir, T_STRING);
+
+  if (RSTRING_LEN(mailpack) < 1) {
+    rb_raise(rb_eArgError, "mailpack is empty");
+  }
+
+  if (RSTRING_LEN(outputdir) < 1) {
+    rb_raise(rb_eArgError, "outputdir is empty");
+  }
+
   fp_err = tmpfile();
 
   if (fp_err == NULL) {
@@ -34,8 +45,21 @@ static VALUE ruby_ripmime_decode(VALUE self, VALUE mailpack, VALUE outputdir) {
 
   RIPMIME_init(&rm);
 
+  // XXX: for strdup memory leak (ripMIME 1.4.0.10)
+  rm.outputdir = NULL;
+  rm.mailpack = NULL;
+
   ret = RIPMIME_decode(&rm, StringValuePtr(mailpack), StringValuePtr(outputdir));
 
+  // XXX: for strdup memory leak (ripMIME 1.4.0.10)
+  if (rm.outputdir) {
+    free(rm.outputdir);
+  }
+
+  if (rm.mailpack) {
+    free(rm.mailpack);
+  }
+
   if (dup2(fd_stdout, fileno(stdout)) == -1) {
     rb_raise(rb_eRuntimeError, strerror(errno));
   }

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ruby-ripmime
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 31
   prerelease: 
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - winebarrel