Skip to content
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.

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

[BUG] Memory leak in BigInt::getString() method. #22

Open
shude opened this issue Jul 10, 2021 · 0 comments
Open

[BUG] Memory leak in BigInt::getString() method. #22

shude opened this issue Jul 10, 2021 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@shude
Copy link

shude commented Jul 10, 2021

Hi, i found memory leak in BigInt::getString() method.
Here is a possible solution:

std::string BigInt::getString(int base) const {
    char *str = mpz_get_str(nullptr, base, data); // mpz_get_str  allocated memory - it must be free 
    std::string result = std::string(str);
    free(str);
    return result;
}
@EndrII EndrII self-assigned this Nov 1, 2021
@EndrII EndrII added the bug Something isn't working label Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants