src: fix issue with Windows using separate heap for node addons

Passing the callback ensures we do the free() inside
the addon, where the heap is the same.

Author: mscdex

.github/workflows/ci.yml

@@ -90,3 +90,8 @@ jobs:
         run: npm install
       - name: Run tests
         run: npm test
+      - name: debug
+        if: ${{ failure() }}
+        uses: mxschmitt/action-tmate@v3
+        with:
+          limit-access-to-actor: true

src/binding.cc

@@ -51,6 +51,10 @@ typedef struct {
   void* val;
 } RowValue;
 
+void free_blob(char* data, void* hint) {
+  free(data);
+}
+
 class BindValueBlob {
   public:
     BindValueBlob(Local<Value>& buf_) {
@@ -806,6 +810,11 @@ void QueryAfter(uv_work_t* req, int status) {
                   val = Nan::NewBuffer(
                     static_cast<char*>(query_req->results[i][j][k].val),
                     query_req->results[i][j][k].len
+#ifdef _MSC_VER
+                    ,
+                    free_blob,
+                    nullptr
+#endif
                   ).ToLocalChecked();
                   break;
                 }