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

Re: [Xen-devel] PATCH 3/4] x86/mem_sharing: drop unused variable.



> 
> Coverity CID 1087198
> 
> Signed-off-by: Tim Deegan <tim@xxxxxxx>
Acked-by: Andres Lagar-Cavilla <andres@xxxxxxxxxxxxxxxx>

Thanks
Andres
> ---
> xen/arch/x86/mm/mem_sharing.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c
> index 1e89f6c..8355f94 100644
> --- a/xen/arch/x86/mm/mem_sharing.c
> +++ b/xen/arch/x86/mm/mem_sharing.c
> @@ -833,7 +833,6 @@ int mem_sharing_nominate_page(struct domain *d,
>     mfn_t mfn;
>     struct page_info *page = NULL; /* gcc... */
>     int ret;
> -    struct gfn_info *gfn_info;
> 
>     *phandle = 0UL;
> 
> @@ -892,7 +891,7 @@ int mem_sharing_nominate_page(struct domain *d,
>     page->sharing->handle = get_next_handle();  
> 
>     /* Create the local gfn info */
> -    if ( (gfn_info = mem_sharing_gfn_alloc(page, d, gfn)) == NULL )
> +    if ( mem_sharing_gfn_alloc(page, d, gfn) == NULL )
>     {
>         xfree(page->sharing);
>         page->sharing = NULL;
> -- 
> 1.8.4.rc3


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.