Hi Serge,

>>> The freeing seems unnecessary as you're about to rexit(-1), right?
>>
>> in the current flow, yes. However, this function may be useful from
>> other places where one does not exit if it fails, so I'd rather be a bit
>> defensive in the programming style, if possible.
> 
> In that case should you also free if the clearenv() fails?

Oh, yes, I missed that. I'll can post a follow-up patch or resend,
whatever is more convenient for you.

-- Christian

------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead. 
Download for free and get started troubleshooting in minutes. 
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to