[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [lvm-devel] LVM2 ./WHATS_NEW lib/activate/activate.h lib/a ...



Dne 14.5.2010 11:00, Milan Broz napsal(a):
> On 05/14/2010 01:47 AM, Alasdair G Kergon wrote:
>> Mixing up types there.
>> name is a dm device name
>>
>>   
> yep. ok with that patch? (we have no mempool available there, that's why
> dm_strdup)
> 
> diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c
> index 710b3ae..d01bd63 100644
> --- a/lib/activate/dev_manager.c
> +++ b/lib/activate/dev_manager.c
> @@ -133,7 +133,7 @@ int device_is_usable(struct device *dev)

>  	r = 1;
>  
>        out:
> +	if (vgname)

- this check is not really needed and done by dm_free itself.

> +		dm_free(vgname);

Zdenek


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]