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
Free baton on error processing javascript arguments 
We only free baton on some cases, but we should try to free all previously allocated memory.
  • Loading branch information
@julianmesa-gitkraken
julianmesa-gitkraken committed Oct 26, 2022
commit 7b5c85b7e063cb4e305364a2cf7474bf9490a4b0
5 changes: 4 additions & 1 deletion generate/templates/partials/async_function.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@ NAN_METHOD({{ cppClassName }}::{{ cppFunctionName }}) {
);

if (!conversionResult.result) {
delete[] baton->{{ arg.name }};
// TODO free previously allocated memory
free(baton->{{ arg.name }});
delete baton;
return Nan::ThrowError(Nan::New(conversionResult.error).ToLocalChecked());
}

Expand All @@ -77,6 +79,7 @@ NAN_METHOD({{ cppClassName }}::{{ cppFunctionName }}) {
{
auto conversionResult = Configurable{{ arg.cppClassName }}::fromJavascript(nodegitContext, info[{{ arg.jsArg }}]);
if (!conversionResult.result) {
delete baton;
return Nan::ThrowError(Nan::New(conversionResult.error).ToLocalChecked());
}

Expand Down