Ard:
   Thanks!  Other two patches are good. Reviewed-by: Liming Gao 
<liming....@intel.com>

Thanks
Liming
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard 
Biesheuvel
Sent: Tuesday, September 20, 2016 8:41 PM
To: Wu, Hao A <hao.a...@intel.com>
Cc: edk2-devel@lists.01.org; Gao, Liming <liming....@intel.com>; 
vish...@qti.qualcomm.com; leif.lindh...@linaro.org
Subject: Re: [edk2] [PATCH v2 2/3] MdePkg/BaseMemoryLib*: add missing ASSERT()s

On 20 September 2016 at 13:16, Ard Biesheuvel wrote:
> On 20 September 2016 at 13:02, Wu, Hao A wrote:
>>> -----Original Message-----
>>> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ard
>>> Biesheuvel
>>> Sent: Tuesday, September 20, 2016 4:38 PM
>>> To: Wu, Hao A
>>> Cc: leif.lindh...@linaro.org<mailto:leif.lindh...@linaro.org>; 
>>> edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>;
>>> vish...@qti.qualcomm.com<mailto:vish...@qti.qualcomm.com>; Gao, Liming
>>> Subject: Re: [edk2] [PATCH v2 2/3] MdePkg/BaseMemoryLib*: add missing
>>> ASSERT()s
>>>
>>> On 20 September 2016 at 03:00, Wu, Hao A wrote:
>>> > Hi Ard,
>>> >
>>> > The NULL checks for the input Guids in APIs CopyGuid(), CompareGuid() and
>>> > IsZeroGuid() are implicitly done within calls to BaseLib APIs
>>> > ReadUnaligned64() and WriteUnaligned64().
>>> >
>>> > So I think the functions behavior matches with their comments. What do you
>>> > think?
>>> >
>>>
>>> I disagree. ReadUnaligned64 and WriteUnaligned64 could theoretically
>>> be implemented by a version of BaseLib that does not contain such
>>> ASSERT()s
>>
>> The comments for APIs ReadUnaligned64 and WriteUnaligned64 in BaseLib
>> mention the ASSERT() for inputting a NULL buffer.
>>
>> I think instances of BaseLib should follow the comments.
>>
>
> I agree with this. But that does not justify omitting them here.

... but actually, it makes no sense to argue about this, since the
user visible outcome is the same. I will withdraw the patch.

Thanks,
Ard.
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to