Merge pull request #1090 from smith-kyle/mem-safety-time

Memory management: duplicate time

generate/input/descriptor.json

@@ -2453,6 +2453,9 @@
         }
       }
     },
+    "time": {
+      "dupFunction": "git_time_dup"
+    },
     "trace": {
       "functions": {
         "git_trace_set": {

generate/templates/manual/include/functions/copy.h

@@ -15,6 +15,7 @@ const git_time *git_time_dup(const git_time *arg);
 const git_diff_delta *git_diff_delta_dup(const git_diff_delta *arg);
 const git_diff_file *git_diff_file_dup(const git_diff_file *arg);
 
+void git_time_dup(git_time **out, const git_time *arg);
 void git_transfer_progress_dup(git_transfer_progress **out, const git_transfer_progress *arg);
 
 #endif

generate/templates/manual/src/functions/copy.cc

@@ -11,6 +11,11 @@ const git_error *git_error_dup(const git_error *arg) {
   return result;
 }
 
+void git_time_dup(git_time **out, const git_time *arg) {
+  *out = (git_time *)malloc(sizeof(git_time));
+  memcpy(*out, arg, sizeof(git_time));
+}
+
 void git_transfer_progress_dup(git_transfer_progress **out, const git_transfer_progress *arg) {
   *out = (git_transfer_progress *)malloc(sizeof(git_transfer_progress));
   memcpy(*out, arg, sizeof(git_transfer_progress));

test/tests/signature.js

@@ -2,6 +2,7 @@ var assert = require("assert");
 var path = require("path");
 var local = path.join.bind(path, __dirname);
 
+var garbageCollect = require("../utils/garbage_collect.js");
 var exec = require("../../utils/execPromise");
 
 describe("Signature", function() {
@@ -83,4 +84,26 @@ describe("Signature", function() {
       });
     });
   });
+
+  it("duplicates time", function() {
+    garbageCollect();
+    var Time = NodeGit.Time;
+    var startSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    var startNonSelfFreeingCount =
+      Time.getNonSelfFreeingConstructedCount();
+    var time = Signature.now(name, email).when();
+
+    garbageCollect();
+    var endSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    var endNonSelfFreeingCount = Time.getNonSelfFreeingConstructedCount();
+    // we should get one duplicated, self-freeing time
+    assert.equal(startSelfFreeingCount + 1, endSelfFreeingCount);
+    assert.equal(startNonSelfFreeingCount, endNonSelfFreeingCount);
+
+    time = null;
+    garbageCollect();
+    endSelfFreeingCount = Time.getSelfFreeingInstanceCount();
+    // the self-freeing time should get freed
+    assert.equal(startSelfFreeingCount, endSelfFreeingCount);
+  });
 });