search

brianc / node-libpq

Simple, low level native bindings to PostgreSQL's libpq from node.js
111 stars 41 forks source link

Fix memory leak #77

Closed scalm closed 2 months ago

scalm commented 4 years ago

I found a memory leak in the latest release. Tested: NodeJS v10.18.1, libpq 1.8.8 - No memory leak NodeJS v10.18.1, libpq 1.8.9 - There is a memory leak NodeJS v12.16.3, libpq 1.8.9 - There is a memory leak

I consulted the documentation and reference tests (https://github.com/nodejs/nan/blob/2c4ee8a32a299eada3cd6e468bbd0a473bfea96d/test/cpp/weak.cpp#L23 ) and tried to fix.

brianc commented 2 months ago

This has been fixed - thanks!