Re: [Xen-devel] [PATCH v2 14/15] xen/xsm: Add support for device tree

[Julien Grall <julien.grall@xxxxxxxxxx>]

Hi Andrew and Ian,

On 03/18/2014 05:34 PM, Andrew Cooper wrote:
> On 18/03/14 17:24, Ian Campbell wrote:
>> On Mon, 2014-03-17 at 14:06 +0000, Julien Grall wrote:
>>> This patch adds a new module "xen,xsm-blob" to allow the user to load the 
>>> XSM
>>> policy when Xen is booting.
>>>
>>> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
>>> Acked-by: Daniel De Graaf <dgdegra@xxxxxxxxxxxxx>
>> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>>> +
>>> +    if ( policy_buffer )
>>> +        xfree(policy_buffer);
>> Is xfree(NULL) not ok in the same way free(NULL) is?
> 
> xfree(NULL) is perfectly fine.  The code should be tweaked.

I will remove the if in the next version.

Regards,

-- 
Julien Grall

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