[edk2-devel] [edk2-libc Patch 1/1] edk2-libc/StdLib: Uninitialized global variable

[Jayaprakash, N]

Reviewed by : Jayaprakash N <n.jayaprakash at intel.com>

-----Original Message-----
From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Jayaprakash, N
Sent: Friday, July 21, 2023 9:22 PM
To: Kinney, Michael D <michael.d.kinney at intel.com>; devel at edk2.groups.io
Cc: Rebecca Cran <rebecca at nuviainc.com>; Kloper, Dimitry <dimitry.kloper at intel.com>; Kloper, Dimitry <dimitry.kloper at intel.com>
Subject: Re: [edk2-devel] [edk2-libc Patch 1/1] edk2-libc/StdLib: Uninitialized global variable

Hi Mike,

I was trying to explore with Co-authored-by tag as I simplified the patch with minor edits.
If it doesn't work I shall remove it.

I noticed after sending the patch. So I have sent V2 patch by removing the , in the names. 

I will go ahead and merge this change.

Regards,
JP
-----Original Message-----
From: Kinney, Michael D <michael.d.kinney at intel.com>
Sent: Friday, July 21, 2023 9:03 PM
To: Jayaprakash, N <n.jayaprakash at intel.com>; devel at edk2.groups.io
Cc: Rebecca Cran <rebecca at nuviainc.com>; Kloper, Dimitry <dimitry.kloper at intel.com>; Kloper, Dimitry <dimitry.kloper at intel.com>; Kinney, Michael D <michael.d.kinney at intel.com>
Subject: RE: [edk2-libc Patch 1/1] edk2-libc/StdLib: Uninitialized global variable

Hi JP,

I have not seen co-authored-by tag used before.

If Dimitry is the author, then please update git commit so Dimitry is the author and remove that tag.

Also, please make sure that names do not contain ','.

Should be: Dimity Kloper <dimitry.kloper at intel.com>

Thanks,

Mike

> -----Original Message-----
> From: Jayaprakash, N <n.jayaprakash at intel.com>
> Sent: Friday, July 21, 2023 8:27 AM
> To: devel at edk2.groups.io
> Cc: Jayaprakash, N <n.jayaprakash at intel.com>; Rebecca Cran 
> <rebecca at nuviainc.com>; Kinney, Michael D 
> <michael.d.kinney at intel.com>; Kloper; Kloper, Dimitry 
> <dimitry.kloper at intel.com>
> Subject: [edk2-libc Patch 1/1] edk2-libc/StdLib: Uninitialized global 
> variable
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4506
> 
> res_init() is called from different places in sockets library. It 
> depends on global _res variable containing a state. The problem is 
> that if __BIND_RES_TEXT macro is not defined, _res is not initialized.
> Depending on compiler and build optimization this can fill the 
> variable with garbage that is later used by res_init().
> Fix is trivial - explicitly initialize _res.
> 
> Cc: Rebecca Cran <rebecca at nuviainc.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Co-authored-by: Kloper, Dimitry <dimitry.kloper at intel.com>
> Signed-off-by: Jayaprakash N <n.jayaprakash at intel.com>
> ---
>  StdLib/BsdSocketLib/res_init.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/StdLib/BsdSocketLib/res_init.c 
> b/StdLib/BsdSocketLib/res_init.c index 613a76a..fbc53c5 100644
> --- a/StdLib/BsdSocketLib/res_init.c
> +++ b/StdLib/BsdSocketLib/res_init.c
> @@ -121,11 +121,11 @@ static u_int32_t net_mask __P((struct in_addr));
>   */
> 
>  struct __res_state _res
> -# if defined(__BIND_RES_TEXT)
> -    = { RES_TIMEOUT, }  /* Motorola, et al. */
> -# endif
> -    ;
> -
> +#if defined(__BIND_RES_TEXT)
> +    = { RES_TIMEOUT, };  /* Motorola, et al. */ #else
> +    = {0};
> +#endif
> 
>  /*
>   * Set up default settings.  If the configuration file exist, the 
> values
> --
> 2.40.0.windows.1








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#107132): https://edk2.groups.io/g/devel/message/107132
Mute This Topic: https://groups.io/mt/100278658/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-