> -----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; edk2-devel@lists.01.org; > 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 <hao.a...@intel.com> 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. > _______________________________________________ > edk2-devel mailing list > 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