Re: [Xen-devel] [PATCH 2/4] xen/arm: initialize access

[Julien Grall <julien.grall@xxxxxxx>]

Hi,

Please use scripts/get_maintainers.pl to CC relevant maintainers. In 
this case you need to add Ravzan and Tamas.

On 15/10/2018 10:56, Stefano Stabellini wrote:
> Initialize variable *access before returning it back to the caller.
Same as the previous patch, why do you need this?

Cheers,

> Signed-off-by: Stefano Stabellini <stefanos@xxxxxxxxxx>
> ---
>   xen/arch/arm/mem_access.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/xen/arch/arm/mem_access.c b/xen/arch/arm/mem_access.c
> index ba4ec78..10ab308 100644
> --- a/xen/arch/arm/mem_access.c
> +++ b/xen/arch/arm/mem_access.c
> @@ -47,6 +47,7 @@ static int __p2m_get_mem_access(struct domain *d, gfn_t gfn,
>       };
>   
>       ASSERT(p2m_is_locked(p2m));
> +    *access = XENMEM_access_n;
>   
>       /* If no setting was ever set, just return rwx. */
>       if ( !p2m->mem_access_enabled )
--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel