----- Original Message -----

On 11/02/11 01:25, NeilBrown wrote:

quoted

Thanks Jes.
I've applied all of these - using the second version of patch 19.

I'm not normally very fussed about freeing and closing things
shortly before
the process is going to exit anyway, but I'm not against it.  And
some of
your patches were for other leaks.

Hi Neil,

I totally agree here. The main reason for applying those fixes is to
reduce the S/N ratio from the checking tools.

It has the advantage that if mdadm is ever converted to run any of
these routines from a long standing operation (say monitor mode decides
to start doing new things), then we aren't tasked with fixing up
1001 little details that could have been right in the first place ;-)

-- 
Doug Ledford [off-list ref]
              GPG KeyID: 0E572FDD
	      http://people.redhat.com/dledford