[Patch] fix small memleak in _dbus_setenv()

Christian Ehrlicher Ch.Ehrlicher at gmx.de
Wed Apr 23 12:49:23 PDT 2008


Havoc Pennington schrieb:
> Hi,
> 
> You can't fix this leak; putenv doesn't copy the string. There should
> be a comment in the code about this, if not, a good patch would be to
> add one.
> 
> This is just a bug in the putenv() API design, which is why setenv and
> unsetenv are preferred.
> 
I don't see such a comment in msdn - so at least it works correct on 
windows.


Christian


More information about the dbus mailing list