* [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
@ 2023-07-22 8:18 Nickle Wang via groups.io
2023-07-22 12:27 ` Mike Maslenkin
2023-07-24 1:15 ` Igor Kulchytskyy via groups.io
0 siblings, 2 replies; 6+ messages in thread
From: Nickle Wang via groups.io @ 2023-07-22 8:18 UTC (permalink / raw)
To: devel; +Cc: Abner Chang, Igor Kulchytskyy, Nick Ramirez
There are two definitions for strncpy() function in RedfishCrtLib.h
Signed-off-by: Nickle Wang <nicklew@nvidia.com>
Cc: Abner Chang <abner.chang@amd.com>
Cc: Igor Kulchytskyy <igork@ami.com>
Cc: Nick Ramirez <nramirez@nvidia.com>
---
RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h b/RedfishPkg/Include/Library/RedfishCrtLib.h
index 23c6acfca33e..a2c17d1b03fc 100644
--- a/RedfishPkg/Include/Library/RedfishCrtLib.h
+++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
@@ -3,6 +3,7 @@
Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
(C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
+ Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent
@@ -304,8 +305,7 @@ char *
strncpy (
char *,
size_t,
- const char *,
- size_t
+ const char *
);
int
--
2.17.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107139): https://edk2.groups.io/g/devel/message/107139
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
2023-07-22 8:18 [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy Nickle Wang via groups.io
@ 2023-07-22 12:27 ` Mike Maslenkin
2023-08-03 6:07 ` Nickle Wang via groups.io
[not found] ` <1777CA85396897C0.28206@groups.io>
2023-07-24 1:15 ` Igor Kulchytskyy via groups.io
1 sibling, 2 replies; 6+ messages in thread
From: Mike Maslenkin @ 2023-07-22 12:27 UTC (permalink / raw)
To: devel, nicklew; +Cc: Abner Chang, Igor Kulchytskyy, Nick Ramirez
There is no usage of strncpy with signature (char *, size_t, char *).
I would suggest to remove this declaration at all.
BTW there are duplicated declaration for memcmp, memset, strncmp.
On Sat, Jul 22, 2023 at 11:18 AM Nickle Wang via groups.io
<nicklew=nvidia.com@groups.io> wrote:
>
> There are two definitions for strncpy() function in RedfishCrtLib.h
>
> Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> Cc: Abner Chang <abner.chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> Cc: Nick Ramirez <nramirez@nvidia.com>
> ---
> RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h b/RedfishPkg/Include/Library/RedfishCrtLib.h
> index 23c6acfca33e..a2c17d1b03fc 100644
> --- a/RedfishPkg/Include/Library/RedfishCrtLib.h
> +++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
> @@ -3,6 +3,7 @@
>
> Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> (C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -304,8 +305,7 @@ char *
> strncpy (
> char *,
> size_t,
> - const char *,
> - size_t
> + const char *
> );
>
> int
> --
> 2.17.1
>
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107143): https://edk2.groups.io/g/devel/message/107143
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
2023-07-22 8:18 [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy Nickle Wang via groups.io
2023-07-22 12:27 ` Mike Maslenkin
@ 2023-07-24 1:15 ` Igor Kulchytskyy via groups.io
2023-07-24 2:03 ` Chang, Abner via groups.io
1 sibling, 1 reply; 6+ messages in thread
From: Igor Kulchytskyy via groups.io @ 2023-07-24 1:15 UTC (permalink / raw)
To: Nickle Wang, devel@edk2.groups.io; +Cc: Abner Chang, Nick Ramirez
Reviewed-by: Igor Kulchytskyy <igork@ami.com>
-----Original Message-----
From: Nickle Wang <nicklew@nvidia.com>
Sent: Saturday, July 22, 2023 4:18 AM
To: devel@edk2.groups.io
Cc: Abner Chang <abner.chang@amd.com>; Igor Kulchytskyy <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
Subject: [EXTERNAL] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
**CAUTION: The e-mail below is from an external source. Please exercise caution before opening attachments, clicking links, or following guidance.**
There are two definitions for strncpy() function in RedfishCrtLib.h
Signed-off-by: Nickle Wang <nicklew@nvidia.com>
Cc: Abner Chang <abner.chang@amd.com>
Cc: Igor Kulchytskyy <igork@ami.com>
Cc: Nick Ramirez <nramirez@nvidia.com>
---
RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h b/RedfishPkg/Include/Library/RedfishCrtLib.h
index 23c6acfca33e..a2c17d1b03fc 100644
--- a/RedfishPkg/Include/Library/RedfishCrtLib.h
+++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
@@ -3,6 +3,7 @@
Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
(C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
+ Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent
@@ -304,8 +305,7 @@ char *
strncpy (
char *,
size_t,
- const char *,
- size_t
+ const char *
);
int
--
2.17.1
-The information contained in this message may be confidential and proprietary to American Megatrends (AMI). This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited. Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107149): https://edk2.groups.io/g/devel/message/107149
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
2023-07-24 1:15 ` Igor Kulchytskyy via groups.io
@ 2023-07-24 2:03 ` Chang, Abner via groups.io
0 siblings, 0 replies; 6+ messages in thread
From: Chang, Abner via groups.io @ 2023-07-24 2:03 UTC (permalink / raw)
To: Igor Kulchytskyy, Nickle Wang, devel@edk2.groups.io; +Cc: Nick Ramirez
[AMD Official Use Only - General]
Reviewed-by: Abner Chang <abner.chang@amd.com>
> -----Original Message-----
> From: Igor Kulchytskyy <igork@ami.com>
> Sent: Monday, July 24, 2023 9:16 AM
> To: Nickle Wang <nicklew@nvidia.com>; devel@edk2.groups.io
> Cc: Chang, Abner <Abner.Chang@amd.com>; Nick Ramirez
> <nramirez@nvidia.com>
> Subject: RE: [EXTERNAL] [PATCH] RedfishPkg/RedfishCrtLib: multiple
> definitions of strncpy.
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Reviewed-by: Igor Kulchytskyy <igork@ami.com>
>
> -----Original Message-----
> From: Nickle Wang <nicklew@nvidia.com>
> Sent: Saturday, July 22, 2023 4:18 AM
> To: devel@edk2.groups.io
> Cc: Abner Chang <abner.chang@amd.com>; Igor Kulchytskyy
> <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
> Subject: [EXTERNAL] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions
> of strncpy.
>
>
> **CAUTION: The e-mail below is from an external source. Please exercise
> caution before opening attachments, clicking links, or following guidance.**
>
> There are two definitions for strncpy() function in RedfishCrtLib.h
>
> Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> Cc: Abner Chang <abner.chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>
> Cc: Nick Ramirez <nramirez@nvidia.com>
> ---
> RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h
> b/RedfishPkg/Include/Library/RedfishCrtLib.h
> index 23c6acfca33e..a2c17d1b03fc 100644
> --- a/RedfishPkg/Include/Library/RedfishCrtLib.h
> +++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
> @@ -3,6 +3,7 @@
>
> Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> (C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights
> reserved.
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> @@ -304,8 +305,7 @@ char *
> strncpy (
> char *,
> size_t,
> - const char *,
> - size_t
> + const char *
> );
>
> int
> --
> 2.17.1
>
> -The information contained in this message may be confidential and
> proprietary to American Megatrends (AMI). This communication is intended
> to be read only by the individual or entity to whom it is addressed or by their
> designee. If the reader of this message is not the intended recipient, you are
> on notice that any distribution of this message, in any form, is strictly
> prohibited. Please promptly notify the sender by reply e-mail or by telephone
> at 770-246-8600, and then delete or destroy all copies of the transmission.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107152): https://edk2.groups.io/g/devel/message/107152
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
2023-07-22 12:27 ` Mike Maslenkin
@ 2023-08-03 6:07 ` Nickle Wang via groups.io
[not found] ` <1777CA85396897C0.28206@groups.io>
1 sibling, 0 replies; 6+ messages in thread
From: Nickle Wang via groups.io @ 2023-08-03 6:07 UTC (permalink / raw)
To: Mike Maslenkin, devel@edk2.groups.io, Abner Chang
Cc: Igor Kulchytskyy, Nick Ramirez
Thanks for your review, Mike!
I am ok to remove all duplicated declarations in this file. @Abner Chang, any concern?
Thanks,
Nickle
> -----Original Message-----
> From: Mike Maslenkin <mike.maslenkin@gmail.com>
> Sent: Saturday, July 22, 2023 8:28 PM
> To: devel@edk2.groups.io; Nickle Wang <nicklew@nvidia.com>
> Cc: Abner Chang <abner.chang@amd.com>; Igor Kulchytskyy <igork@ami.com>;
> Nick Ramirez <nramirez@nvidia.com>
> Subject: Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions
> of strncpy.
>
> External email: Use caution opening links or attachments
>
>
> There is no usage of strncpy with signature (char *, size_t, char *).
> I would suggest to remove this declaration at all.
> BTW there are duplicated declaration for memcmp, memset, strncmp.
>
> On Sat, Jul 22, 2023 at 11:18 AM Nickle Wang via groups.io
> <nicklew=nvidia.com@groups.io> wrote:
> >
> > There are two definitions for strncpy() function in RedfishCrtLib.h
> >
> > Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> > Cc: Abner Chang <abner.chang@amd.com>
> > Cc: Igor Kulchytskyy <igork@ami.com>
> > Cc: Nick Ramirez <nramirez@nvidia.com>
> > ---
> > RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h
> > b/RedfishPkg/Include/Library/RedfishCrtLib.h
> > index 23c6acfca33e..a2c17d1b03fc 100644
> > --- a/RedfishPkg/Include/Library/RedfishCrtLib.h
> > +++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
> > @@ -3,6 +3,7 @@
> >
> > Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> > (C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> > + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
> >
> > SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > @@ -304,8 +305,7 @@ char *
> > strncpy (
> > char *,
> > size_t,
> > - const char *,
> > - size_t
> > + const char *
> > );
> >
> > int
> > --
> > 2.17.1
> >
> >
> >
> >
> >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107518): https://edk2.groups.io/g/devel/message/107518
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy.
[not found] ` <1777CA85396897C0.28206@groups.io>
@ 2023-10-23 14:20 ` Nickle Wang via groups.io
0 siblings, 0 replies; 6+ messages in thread
From: Nickle Wang via groups.io @ 2023-10-23 14:20 UTC (permalink / raw)
To: devel@edk2.groups.io, Nickle Wang, Mike Maslenkin, Abner Chang
Cc: Igor Kulchytskyy, Nick Ramirez
Hi Mike,
Sorry for the delay. I remove all duplicated definitions in RedfishCrtLib.h. Please check new patch file here: https://edk2.groups.io/g/devel/message/109925
Thanks,
Nickle
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Nickle Wang
> via groups.io
> Sent: Thursday, August 3, 2023 2:08 PM
> To: Mike Maslenkin <mike.maslenkin@gmail.com>; devel@edk2.groups.io; Abner
> Chang <abner.chang@amd.com>
> Cc: Igor Kulchytskyy <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
> Subject: Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions
> of strncpy.
>
> External email: Use caution opening links or attachments
>
>
> Thanks for your review, Mike!
>
> I am ok to remove all duplicated declarations in this file. @Abner Chang, any
> concern?
>
> Thanks,
> Nickle
>
> > -----Original Message-----
> > From: Mike Maslenkin <mike.maslenkin@gmail.com>
> > Sent: Saturday, July 22, 2023 8:28 PM
> > To: devel@edk2.groups.io; Nickle Wang <nicklew@nvidia.com>
> > Cc: Abner Chang <abner.chang@amd.com>; Igor Kulchytskyy
> > <igork@ami.com>; Nick Ramirez <nramirez@nvidia.com>
> > Subject: Re: [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple
> > definitions of strncpy.
> >
> > External email: Use caution opening links or attachments
> >
> >
> > There is no usage of strncpy with signature (char *, size_t, char *).
> > I would suggest to remove this declaration at all.
> > BTW there are duplicated declaration for memcmp, memset, strncmp.
> >
> > On Sat, Jul 22, 2023 at 11:18 AM Nickle Wang via groups.io
> > <nicklew=nvidia.com@groups.io> wrote:
> > >
> > > There are two definitions for strncpy() function in RedfishCrtLib.h
> > >
> > > Signed-off-by: Nickle Wang <nicklew@nvidia.com>
> > > Cc: Abner Chang <abner.chang@amd.com>
> > > Cc: Igor Kulchytskyy <igork@ami.com>
> > > Cc: Nick Ramirez <nramirez@nvidia.com>
> > > ---
> > > RedfishPkg/Include/Library/RedfishCrtLib.h | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/RedfishPkg/Include/Library/RedfishCrtLib.h
> > > b/RedfishPkg/Include/Library/RedfishCrtLib.h
> > > index 23c6acfca33e..a2c17d1b03fc 100644
> > > --- a/RedfishPkg/Include/Library/RedfishCrtLib.h
> > > +++ b/RedfishPkg/Include/Library/RedfishCrtLib.h
> > > @@ -3,6 +3,7 @@
> > >
> > > Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> > > (C) Copyright 2021 Hewlett Packard Enterprise Development LP<BR>
> > > + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights
> reserved.
> > >
> > > SPDX-License-Identifier: BSD-2-Clause-Patent
> > >
> > > @@ -304,8 +305,7 @@ char *
> > > strncpy (
> > > char *,
> > > size_t,
> > > - const char *,
> > > - size_t
> > > + const char *
> > > );
> > >
> > > int
> > > --
> > > 2.17.1
> > >
> > >
> > >
> > >
> > >
> > >
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#109928): https://edk2.groups.io/g/devel/message/109928
Mute This Topic: https://groups.io/mt/100292360/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-10-23 14:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-22 8:18 [edk2-devel] [PATCH] RedfishPkg/RedfishCrtLib: multiple definitions of strncpy Nickle Wang via groups.io
2023-07-22 12:27 ` Mike Maslenkin
2023-08-03 6:07 ` Nickle Wang via groups.io
[not found] ` <1777CA85396897C0.28206@groups.io>
2023-10-23 14:20 ` Nickle Wang via groups.io
2023-07-24 1:15 ` Igor Kulchytskyy via groups.io
2023-07-24 2:03 ` Chang, Abner via groups.io
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox