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

memory leak in main_client #206

Open
pascal-niklaus opened this issue Apr 7, 2024 · 1 comment
Open

memory leak in main_client #206

pascal-niklaus opened this issue Apr 7, 2024 · 1 comment
Labels

Comments

@pascal-niklaus
Copy link
Contributor

I will at some stage prepare a PR to fix this, but want to document the issue here as a reminder

Describe the bug

In main.c, function main_client, clientdata is passed to the server.

  • sometimes this a a part of argv, which does not need to be freed

  • however, for the line argument, g_strjoin allocates a string that IMO is never freed

@bk138
Copy link
Owner

bk138 commented Apr 7, 2024

ACK, should be a strdup if part of argv and freed accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants