U++ forum: Welcome to the forum

[BUG] Heap leak in SysExec package [message #41833]

Mon, 27 January 2014 18:36

masu
Messages: 378
Registered: February 2006

Senior Member

Hi,

there is a heap leak in SysExec.cpp originating from not freeing argv and envv pointers at the end of SysExec function.

In addition I propose to introduce a flag DELETE_TMP_FILES or something similar to delete created tmp files when SysExec finishes. The reason for this request is that I have a programm that repeatedly calls SysExec and therefore the directory containing tmp files ends up unnecessarily containing a lot of them.

The proposed patch is attached.

[edit] Wherever BuildArgs or BuildEnv are called the pointers need to be freed e.g., also in SudoLib.cpp.

Regards
Matthias

Attachment: SysExec.cpp.patch
(Size: 1.50KB, Downloaded 321 times)

[Updated on: Mon, 27 January 2014 20:37]

Report message to a moderator