[PATCH] [BZ #15381] Initialize wide struct info.
Ondřej Bílka
neleai@seznam.cz
Tue May 14 12:32:00 GMT 2013
On Tue, May 14, 2013 at 01:59:13PM +0200, Julius Plenz wrote:
> Hi,
>
> * Andreas Schwab <schwab@suse.de>:
> > > Are you opposed to initializing _wide_data to NULL?
> >
> > I would prefer -1 so that broken uses crash predictably. :-)
>
> Since undefined behaviour can be anything you like, both initializing
> with NULL and with -1 is equally "valid".
>
OK, here is version with -1 (or 0xdeadbeef if you like it more)
---
libio/genops.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/libio/genops.c b/libio/genops.c
index 390d8d2..18039aa 100644
--- a/libio/genops.c
+++ b/libio/genops.c
@@ -643,6 +643,7 @@ _IO_no_init (fp, flags, orientation, wd, jmp)
{
_IO_old_init (fp, flags);
fp->_mode = orientation;
+ fp->_wide_data = (void *) -1;
#if defined _LIBC || defined _GLIBCPP_USE_WCHAR_T
if (orientation >= 0)
{
--
1.7.4.4
More information about the Libc-alpha
mailing list