Skip to content

Fix malloc return value checking; Fix potential memory leak#123

Merged
troglobit merged 1 commit intolibnet:masterfrom
ivalery111:fix-malloc-check
May 1, 2021
Merged

Fix malloc return value checking; Fix potential memory leak#123
troglobit merged 1 commit intolibnet:masterfrom
ivalery111:fix-malloc-check

Conversation

@ivalery111
Copy link
Contributor

It's seems like a mistake in checking malloc return value. It's may lead to memory leak.
Thank you for review:)

@troglobit
Copy link
Collaborator

Definitely a mistake. Thanks!

We shouldn't use new either, reserved keyword in C++, so if you want you could submit a PR for replacing that too across the tree :)

@troglobit troglobit merged commit c5d560a into libnet:master May 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants