[lvm-devel] [PATCH] use proper pv structure in process_each_segment_in_pv

[Petr Rockai]

Milan Broz <mbroz at redhat.com> writes:
> Use pv from newly read vg to avoid possible unallocated memory use.
Do you maybe mean uninitialised here?

> We need use the new pv from just read-in volume group.
Quite sensible.

> Signed-off-by: Milan Broz <mbroz at redhat.com>
Acked-By: Petr Ročkai <prockai at redhat.com>

> diff --git a/tools/toollib.c b/tools/toollib.c
> index 1a2fd01..aa2194c 100644
> --- a/tools/toollib.c
> +++ b/tools/toollib.c
> @@ -364,6 +364,7 @@ int process_each_segment_in_pv(struct cmd_context *cmd,
>  						      void *handle))
>  {
>  	struct pv_segment *pvseg;
> +	struct pv_list *pvl;
>  	const char *vg_name = NULL;
>  	int ret_max = ECMD_PROCESSED;
>  	int ret;
> @@ -376,6 +377,18 @@ int process_each_segment_in_pv(struct cmd_context *cmd,
>  			log_error("Skipping volume group %s", vg_name);
>  			return ECMD_FAILED;
>  		}
> +
> +		/*
> +		 * Replace possibly incomplete PV structure with new one
> +		 * allocated in vg_read_internal() path.
> +		 */
> +		if (!(pvl = find_pv_in_vg(vg, pv_dev_name(pv)))) {
> +			 log_error("Unable to find %s in volume group %s",
> +				   pv_dev_name(pv), vg_name);
> +			return ECMD_FAILED;
> +		}
> +
> +		pv = pvl->pv;
>  	}
>  
>  	dm_list_iterate_items(pvseg, &pv->segments) {
Check.

-- 
Peter Rockai | me()mornfall!net | prockai()redhat!com
 http://blog.mornfall.net | http://web.mornfall.net

"In My Egotistical Opinion, most people's C programs should be
 indented six feet downward and covered with dirt."
     -- Blair P. Houghton on the subject of C program indentation