Visitar URL original
Fix memory leaks by julianmesa-gitkraken · Pull Request #1944 · nodegit/nodegit · GitHub
Skip to content

Fix memory leaks #1944

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
julianmesa-gitkraken wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Fix memory leaks #1944

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
77cdd5c
Fix memory leaks
julianmesa-gitkraken Oct 20, 2022
5926ba1
Fix git_diff_line leak
julianmesa-gitkraken Oct 20, 2022
2900bf1
Fix leak freeing PatchData
julianmesa-gitkraken Oct 20, 2022
6c4e32d
Fix leak
julianmesa-gitkraken Oct 21, 2022
5160975
Fix leaks git str array
julianmesa-gitkraken Oct 25, 2022
9eb357e
Fix leak strdup in oid
julianmesa-gitkraken Oct 25, 2022
06b04c7
Fix leak in GitPatch::ConvenientFromDiff
julianmesa-gitkraken Oct 25, 2022
c8afbe6
Fix leak freeing array from javascript
julianmesa-gitkraken Oct 25, 2022
54087a6
Fix double freeing
julianmesa-gitkraken Oct 25, 2022
7d85533
Fix leak in GitRevwalk::FileHistoryWalk
julianmesa-gitkraken Oct 25, 2022
07a4593
Fix leak on freeing git_index_entry
julianmesa-gitkraken Oct 25, 2022
7b5c85b
Free baton on error processing javascript arguments
julianmesa-gitkraken Oct 26, 2022
e3fd6fe
Fix leak when the out parameter is not self freeing
julianmesa-gitkraken Oct 26, 2022
7a51839
Fix leak strdup getting oid from string
julianmesa-gitkraken Oct 26, 2022
535395f
Fix leaks in sync functions
julianmesa-gitkraken Oct 26, 2022
ee48c70
Fix leaks in extract_signature template
julianmesa-gitkraken Oct 26, 2022
d4981f0
Fix issues with index entries
julianmesa-gitkraken Oct 28, 2022
842196f
Unregister filters before clean the memory
julianmesa-gitkraken Nov 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix leak 
 We need to free duplicated memory we are responsible for that we obtained from libgit2 because nodegit duplicates it again when calling the wrapper
  • Loading branch information
@julianmesa-gitkraken
julianmesa-gitkraken committed Oct 21, 2022
commit 6c4e32d684ed3fad8f9d437868de884f94d5217f
13 changes: 13 additions & 0 deletions generate/templates/partials/async_function.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -311,6 +311,19 @@ void {{ cppClassName }}::{{ cppFunctionName }}Worker::HandleOKCallback() {
Nan::Set(result, Nan::New("{{ _return.returnNameOrName }}").ToLocalChecked(), v8ConversionSlot);
{%endif%}
{%endeach%}

{%each args|argsInfo as arg %}
{%if not arg.shouldAlloc %}
{%if arg.ownedByThis|and arg.dupFunction|and arg.freeFunctionName|and arg.isReturn|and arg.selfFreeing %}
// We need to free duplicated memory we are responsible for that we obtained from libgit2 because
// nodegit duplicates it again when calling the wrapper
if(baton->{{ arg.name }} != NULL) {
{{ arg.freeFunctionName }}(baton->{{ arg.name }});
}
{%endif%}
{%endif%}
{%endeach%}

{%if .|returnsCount == 1 %}
v8::Local<v8::Value> result = v8ConversionSlot;
{%endif%}
Expand Down