public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH 1/1] CryptoPkg: Fix coding style
@ 2019-08-16  6:53 Zhang, Shenglei
  2019-08-16  7:18 ` Wang, Jian J
  0 siblings, 1 reply; 4+ messages in thread
From: Zhang, Shenglei @ 2019-08-16  6:53 UTC (permalink / raw)
  To: devel; +Cc: Jian Wang, Ting Ye

Update attribute "Out" to "out".
The original "Out" can not pass ECC check.

Cc: Jian Wang <jian.j.wang@intel.com>
Cc: Ting Ye <ting.ye@intel.com>
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
 CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c     | 2 +-
 CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c | 2 +-
 CryptoPkg/Include/Library/BaseCryptLib.h           | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
index f0fcef211d3f..3a827dadfcbd 100644
--- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
+++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
@@ -19,7 +19,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
   @param[in]   SaltSize         Salt size in bytes.
   @param[in]   Info             Pointer to the application specific info.
   @param[in]   InfoSize         Info size in bytes.
-  @param[Out]  Out              Pointer to buffer to receive hkdf value.
+  @param[out]  Out              Pointer to buffer to receive hkdf value.
   @param[in]   OutSize          Size of hkdf bytes to generate.
 
   @retval TRUE   Hkdf generated successfully.
diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
index 73deb5bc3614..19d795a4cc16 100644
--- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
+++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
@@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
   @param[in]   SaltSize         Salt size in bytes.
   @param[in]   Info             Pointer to the application specific info.
   @param[in]   InfoSize         Info size in bytes.
-  @param[Out]  Out              Pointer to buffer to receive hkdf value.
+  @param[out]  Out              Pointer to buffer to receive hkdf value.
   @param[in]   OutSize          Size of hkdf bytes to generate.
 
   @retval TRUE   Hkdf generated successfully.
diff --git a/CryptoPkg/Include/Library/BaseCryptLib.h b/CryptoPkg/Include/Library/BaseCryptLib.h
index da32bb2444fd..8fe303a0b390 100644
--- a/CryptoPkg/Include/Library/BaseCryptLib.h
+++ b/CryptoPkg/Include/Library/BaseCryptLib.h
@@ -3135,7 +3135,7 @@ RandomBytes (
   @param[in]   SaltSize         Salt size in bytes.
   @param[in]   Info             Pointer to the application specific info.
   @param[in]   InfoSize         Info size in bytes.
-  @param[Out]  Out              Pointer to buffer to receive hkdf value.
+  @param[out]  Out              Pointer to buffer to receive hkdf value.
   @param[in]   OutSize          Size of hkdf bytes to generate.
 
   @retval TRUE   Hkdf generated successfully.
-- 
2.18.0.windows.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] CryptoPkg: Fix coding style
  2019-08-16  6:53 [PATCH 1/1] CryptoPkg: Fix coding style Zhang, Shenglei
@ 2019-08-16  7:18 ` Wang, Jian J
  2019-08-16  7:56   ` Zhang, Shenglei
  0 siblings, 1 reply; 4+ messages in thread
From: Wang, Jian J @ 2019-08-16  7:18 UTC (permalink / raw)
  To: Zhang, Shenglei, devel@edk2.groups.io; +Cc: Ye, Ting


Reviewed-by: Jian J Wang <jian.j.wang@intel.com>


> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Friday, August 16, 2019 2:54 PM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Ye, Ting <ting.ye@intel.com>
> Subject: [PATCH 1/1] CryptoPkg: Fix coding style
> 
> Update attribute "Out" to "out".
> The original "Out" can not pass ECC check.
> 
> Cc: Jian Wang <jian.j.wang@intel.com>
> Cc: Ting Ye <ting.ye@intel.com>
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> ---
>  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c     | 2 +-
>  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c | 2 +-
>  CryptoPkg/Include/Library/BaseCryptLib.h           | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> index f0fcef211d3f..3a827dadfcbd 100644
> --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> @@ -19,7 +19,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>    @param[in]   SaltSize         Salt size in bytes.
>    @param[in]   Info             Pointer to the application specific info.
>    @param[in]   InfoSize         Info size in bytes.
> -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>    @param[in]   OutSize          Size of hkdf bytes to generate.
> 
>    @retval TRUE   Hkdf generated successfully.
> diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> index 73deb5bc3614..19d795a4cc16 100644
> --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> @@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>    @param[in]   SaltSize         Salt size in bytes.
>    @param[in]   Info             Pointer to the application specific info.
>    @param[in]   InfoSize         Info size in bytes.
> -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>    @param[in]   OutSize          Size of hkdf bytes to generate.
> 
>    @retval TRUE   Hkdf generated successfully.
> diff --git a/CryptoPkg/Include/Library/BaseCryptLib.h
> b/CryptoPkg/Include/Library/BaseCryptLib.h
> index da32bb2444fd..8fe303a0b390 100644
> --- a/CryptoPkg/Include/Library/BaseCryptLib.h
> +++ b/CryptoPkg/Include/Library/BaseCryptLib.h
> @@ -3135,7 +3135,7 @@ RandomBytes (
>    @param[in]   SaltSize         Salt size in bytes.
>    @param[in]   Info             Pointer to the application specific info.
>    @param[in]   InfoSize         Info size in bytes.
> -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>    @param[in]   OutSize          Size of hkdf bytes to generate.
> 
>    @retval TRUE   Hkdf generated successfully.
> --
> 2.18.0.windows.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/1] CryptoPkg: Fix coding style
  2019-08-16  7:18 ` Wang, Jian J
@ 2019-08-16  7:56   ` Zhang, Shenglei
  2019-08-19  0:48     ` [edk2-devel] " Liming Gao
  0 siblings, 1 reply; 4+ messages in thread
From: Zhang, Shenglei @ 2019-08-16  7:56 UTC (permalink / raw)
  To: Wang, Jian J, devel@edk2.groups.io; +Cc: Ye, Ting

Hi All,

This patch has little impact. So I think it's ok to be checked in before code freeze.

Thanks,
Shenglei

> -----Original Message-----
> From: Wang, Jian J
> Sent: Friday, August 16, 2019 3:18 PM
> To: Zhang, Shenglei <shenglei.zhang@intel.com>; devel@edk2.groups.io
> Cc: Ye, Ting <ting.ye@intel.com>
> Subject: RE: [PATCH 1/1] CryptoPkg: Fix coding style
> 
> 
> Reviewed-by: Jian J Wang <jian.j.wang@intel.com>
> 
> 
> > -----Original Message-----
> > From: Zhang, Shenglei
> > Sent: Friday, August 16, 2019 2:54 PM
> > To: devel@edk2.groups.io
> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ye, Ting <ting.ye@intel.com>
> > Subject: [PATCH 1/1] CryptoPkg: Fix coding style
> >
> > Update attribute "Out" to "out".
> > The original "Out" can not pass ECC check.
> >
> > Cc: Jian Wang <jian.j.wang@intel.com>
> > Cc: Ting Ye <ting.ye@intel.com>
> > Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> > ---
> >  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c     | 2 +-
> >  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c | 2 +-
> >  CryptoPkg/Include/Library/BaseCryptLib.h           | 2 +-
> >  3 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> > b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> > index f0fcef211d3f..3a827dadfcbd 100644
> > --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> > +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
> > @@ -19,7 +19,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> >    @param[in]   SaltSize         Salt size in bytes.
> >    @param[in]   Info             Pointer to the application specific info.
> >    @param[in]   InfoSize         Info size in bytes.
> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
> >    @param[in]   OutSize          Size of hkdf bytes to generate.
> >
> >    @retval TRUE   Hkdf generated successfully.
> > diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> > b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> > index 73deb5bc3614..19d795a4cc16 100644
> > --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> > +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
> > @@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> >    @param[in]   SaltSize         Salt size in bytes.
> >    @param[in]   Info             Pointer to the application specific info.
> >    @param[in]   InfoSize         Info size in bytes.
> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
> >    @param[in]   OutSize          Size of hkdf bytes to generate.
> >
> >    @retval TRUE   Hkdf generated successfully.
> > diff --git a/CryptoPkg/Include/Library/BaseCryptLib.h
> > b/CryptoPkg/Include/Library/BaseCryptLib.h
> > index da32bb2444fd..8fe303a0b390 100644
> > --- a/CryptoPkg/Include/Library/BaseCryptLib.h
> > +++ b/CryptoPkg/Include/Library/BaseCryptLib.h
> > @@ -3135,7 +3135,7 @@ RandomBytes (
> >    @param[in]   SaltSize         Salt size in bytes.
> >    @param[in]   Info             Pointer to the application specific info.
> >    @param[in]   InfoSize         Info size in bytes.
> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
> >    @param[in]   OutSize          Size of hkdf bytes to generate.
> >
> >    @retval TRUE   Hkdf generated successfully.
> > --
> > 2.18.0.windows.1


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [edk2-devel] [PATCH 1/1] CryptoPkg: Fix coding style
  2019-08-16  7:56   ` Zhang, Shenglei
@ 2019-08-19  0:48     ` Liming Gao
  0 siblings, 0 replies; 4+ messages in thread
From: Liming Gao @ 2019-08-19  0:48 UTC (permalink / raw)
  To: devel@edk2.groups.io, Zhang, Shenglei, Wang, Jian J; +Cc: Ye, Ting

push@ 944bd5cf1d69741f3705983f650f6adad696a7d1

>-----Original Message-----
>From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
>Zhang, Shenglei
>Sent: Friday, August 16, 2019 3:56 PM
>To: Wang, Jian J <jian.j.wang@intel.com>; devel@edk2.groups.io
>Cc: Ye, Ting <ting.ye@intel.com>
>Subject: Re: [edk2-devel] [PATCH 1/1] CryptoPkg: Fix coding style
>
>Hi All,
>
>This patch has little impact. So I think it's ok to be checked in before code
>freeze.
>
>Thanks,
>Shenglei
>
>> -----Original Message-----
>> From: Wang, Jian J
>> Sent: Friday, August 16, 2019 3:18 PM
>> To: Zhang, Shenglei <shenglei.zhang@intel.com>; devel@edk2.groups.io
>> Cc: Ye, Ting <ting.ye@intel.com>
>> Subject: RE: [PATCH 1/1] CryptoPkg: Fix coding style
>>
>>
>> Reviewed-by: Jian J Wang <jian.j.wang@intel.com>
>>
>>
>> > -----Original Message-----
>> > From: Zhang, Shenglei
>> > Sent: Friday, August 16, 2019 2:54 PM
>> > To: devel@edk2.groups.io
>> > Cc: Wang, Jian J <jian.j.wang@intel.com>; Ye, Ting <ting.ye@intel.com>
>> > Subject: [PATCH 1/1] CryptoPkg: Fix coding style
>> >
>> > Update attribute "Out" to "out".
>> > The original "Out" can not pass ECC check.
>> >
>> > Cc: Jian Wang <jian.j.wang@intel.com>
>> > Cc: Ting Ye <ting.ye@intel.com>
>> > Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
>> > ---
>> >  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c     | 2 +-
>> >  CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c | 2 +-
>> >  CryptoPkg/Include/Library/BaseCryptLib.h           | 2 +-
>> >  3 files changed, 3 insertions(+), 3 deletions(-)
>> >
>> > diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
>> > b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
>> > index f0fcef211d3f..3a827dadfcbd 100644
>> > --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
>> > +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdf.c
>> > @@ -19,7 +19,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>> >    @param[in]   SaltSize         Salt size in bytes.
>> >    @param[in]   Info             Pointer to the application specific info.
>> >    @param[in]   InfoSize         Info size in bytes.
>> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
>> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>> >    @param[in]   OutSize          Size of hkdf bytes to generate.
>> >
>> >    @retval TRUE   Hkdf generated successfully.
>> > diff --git a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
>> > b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
>> > index 73deb5bc3614..19d795a4cc16 100644
>> > --- a/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
>> > +++ b/CryptoPkg/Library/BaseCryptLib/Kdf/CryptHkdfNull.c
>> > @@ -18,7 +18,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>> >    @param[in]   SaltSize         Salt size in bytes.
>> >    @param[in]   Info             Pointer to the application specific info.
>> >    @param[in]   InfoSize         Info size in bytes.
>> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
>> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>> >    @param[in]   OutSize          Size of hkdf bytes to generate.
>> >
>> >    @retval TRUE   Hkdf generated successfully.
>> > diff --git a/CryptoPkg/Include/Library/BaseCryptLib.h
>> > b/CryptoPkg/Include/Library/BaseCryptLib.h
>> > index da32bb2444fd..8fe303a0b390 100644
>> > --- a/CryptoPkg/Include/Library/BaseCryptLib.h
>> > +++ b/CryptoPkg/Include/Library/BaseCryptLib.h
>> > @@ -3135,7 +3135,7 @@ RandomBytes (
>> >    @param[in]   SaltSize         Salt size in bytes.
>> >    @param[in]   Info             Pointer to the application specific info.
>> >    @param[in]   InfoSize         Info size in bytes.
>> > -  @param[Out]  Out              Pointer to buffer to receive hkdf value.
>> > +  @param[out]  Out              Pointer to buffer to receive hkdf value.
>> >    @param[in]   OutSize          Size of hkdf bytes to generate.
>> >
>> >    @retval TRUE   Hkdf generated successfully.
>> > --
>> > 2.18.0.windows.1
>
>
>


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-08-19  0:48 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-08-16  6:53 [PATCH 1/1] CryptoPkg: Fix coding style Zhang, Shenglei
2019-08-16  7:18 ` Wang, Jian J
2019-08-16  7:56   ` Zhang, Shenglei
2019-08-19  0:48     ` [edk2-devel] " Liming Gao

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox