Re: [Freeipa-devel] [PATCHES] Fix some of the issues found by coverity

2011-03-28 Thread Rob Crittenden
Rob Crittenden wrote: Simo Sorce wrote: One is a memory leak that can happen in some error paths. It is not highly probable to happen, so it can be deferred to post GA The other is a uninitialized variable that could cause a segfault in some cases (not seen on the wild, depends on an error path

Re: [Freeipa-devel] [PATCHES] Fix some of the issues found by coverity

2011-03-24 Thread Rob Crittenden
Simo Sorce wrote: One is a memory leak that can happen in some error paths. It is not highly probable to happen, so it can be deferred to post GA The other is a uninitialized variable that could cause a segfault in some cases (not seen on the wild, depends on an error path too). Simo. Ack on

[Freeipa-devel] [PATCHES] Fix some of the issues found by coverity

2011-03-24 Thread Simo Sorce
One is a memory leak that can happen in some error paths. It is not highly probable to happen, so it can be deferred to post GA The other is a uninitialized variable that could cause a segfault in some cases (not seen on the wild, depends on an error path too). Simo. -- Simo Sorce * Red Hat, In