Re: CVS commit: src/lib/libc/stdlib

[christos%zoulas.com@localhost (Christos Zoulas)]

On Sep 24, 12:57pm, tsugutomo.enami%jp.sony.com@localhost (enami tsugutomo) 
wrote:
-- Subject: Re: CVS commit: src/lib/libc/stdlib

| Actual change done in setenv.c rev. 1.33 causes following program
| memory leak (while rev. 1.32 does not).
| 
|       #include <stdlib.h>
| 
|       main()
|       {
| 
|               while (1)
|                       setenv("dummyvar", "dummyval", 1);
|       }
| 
| And rev. 1.34 introduces lock leak on error path while it leaves
| another fixable memory leak.
| 
| I guess following patch is worth to be applied.
| 
| enami.

You are absolutely right!

Thanks,

christos