pd-externals / command

execute system commands from Pd
Other
4 stars 1 forks source link

memleak #5

Closed umlaeute closed 3 years ago

umlaeute commented 3 years ago

it seems that there is a memory leak in https://github.com/pd-externals/command/blob/091fea7d688e4062cf1f6d5aa2a3d0ad90f8deed/command.c#L244

it's probably not a big issue, as the data is allocated in the fork()ed process, which gets destroyed soon enough, but still...

also, 255 characters per argument seem to be a bit low. is there any specific reason to not use MAXPDSTRING?