gromit-mpx icon indicating copy to clipboard operation
gromit-mpx copied to clipboard

memory leak in main_client

Open pascal-niklaus opened this issue 1 year ago • 1 comments

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

pascal-niklaus avatar Apr 07 '24 16:04 pascal-niklaus

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

bk138 avatar Apr 07 '24 18:04 bk138