diff options
| -rw-r--r-- | ChangeLog | 39 | ||||
| -rw-r--r-- | iconv/Versions | 3 | ||||
| -rw-r--r-- | iconv/gconv.h | 48 | ||||
| -rw-r--r-- | iconv/gconv_close.c | 14 | ||||
| -rw-r--r-- | iconv/gconv_int.h | 30 | ||||
| -rw-r--r-- | iconv/gconv_open.c | 152 | ||||
| -rw-r--r-- | iconv/gconv_trans.c | 12 | ||||
| -rw-r--r-- | iconv/loop.c | 14 | ||||
| -rw-r--r-- | iconv/skeleton.c | 14 | ||||
| -rw-r--r-- | libio/fileops.c | 9 | ||||
| -rw-r--r-- | libio/iofwide.c | 18 | ||||
| -rw-r--r-- | wcsmbs/btowc.c | 1 | ||||
| -rw-r--r-- | wcsmbs/mbrtoc16.c | 1 | ||||
| -rw-r--r-- | wcsmbs/mbrtowc.c | 1 | ||||
| -rw-r--r-- | wcsmbs/mbsnrtowcs.c | 1 | ||||
| -rw-r--r-- | wcsmbs/mbsrtowcs_l.c | 1 | ||||
| -rw-r--r-- | wcsmbs/wcrtomb.c | 1 | ||||
| -rw-r--r-- | wcsmbs/wcsnrtombs.c | 1 | ||||
| -rw-r--r-- | wcsmbs/wcsrtombs.c | 1 | ||||
| -rw-r--r-- | wcsmbs/wctob.c | 1 |
20 files changed, 75 insertions, 287 deletions
@@ -1,3 +1,42 @@ +2014-08-12 Florian Weimer <fweimer@redhat.com> + + * iconv/gconv_open.c (__gconv_open): Remove transliteration module + loading. + * iconv/Versions (__gconv_transliterate): Export for use from + gconv modules. + * iconv/gconv.h (__GCONV_TRANSLIT): New flag. + (struct __gconv_trans_data, __gconv_trans_fct, + __gconv_trans_context_fct, __gconv_trans_query_fct, + __gconv_trans_init_fct, __gconv_trans_end_fct): Remove type + definitions. + (struct __gconv_step_data): Remove __trans member. + (__gconv_transliterate): Declaration moved from gconv_int.h. No + longer hidden. Remove unused trans_data argument. + * iconv/gconv_int.h (struct trans_struct): Remove definition. + (__gconv_translit_find): Remove declaration. + (__gconv_transliterate): Declaration moved to gconv.h. Add hidden + prototype. + * iconv/gconv_close.c (__gconv_close): Remove __trans cleanup. + * iconv/gconv_trans.c (__gconv_transliterate): Remove unused + trans_data argument. Add hidden definition. + (__gconv_translit_find): Remove. + * iconv/loop.c (STANDARD_TO_LOOP_ERR_HANDLER): Call + __gconv_transliterate directly if __GCONV_TRANSLIT is set. + * iconv/skeleton.c: Remove transliteration initialization. + * libio/fileops.c (_IO_new_file_fopen): Adjust struct + __gconv_step_data initialization. + * libio/iofwide.c (__libio_translit_): Remove. + (_IO_fwide): Adjust struct __gconv_step_data initialization. + * wcsmbs/btowc.c (__btowc): Likewise. + * wcsmbs/mbrtoc16.c (mbrtoc16): Likewise. + * wcsmbs/mbrtowc.c (__mbrtowc): Likewise. + * wcsmbs/mbsnrtowcs.c (__mbsnrtowcs): Likewise. + * wcsmbs/mbsrtowcs_l.c (__mbsrtowcs_l): Likewise. + * wcsmbs/wcrtomb.c (__wcrtomb): Likewise. + * wcsmbs/wcsnrtombs.c (__wcsnrtombs): Likewise. + * wcsmbs/wcsrtombs.c (__wcsrtombs): Likewise. + * wcsmbs/wctob.c (wctob): Likewise. + 2014-09-12 Siddhesh Poyarekar <siddhesh@redhat.com> [BZ #16194] diff --git a/iconv/Versions b/iconv/Versions index 5d50cf11e2..60ab10a277 100644 --- a/iconv/Versions +++ b/iconv/Versions @@ -6,5 +6,8 @@ libc { GLIBC_PRIVATE { # functions shared with iconv program __gconv_get_alias_db; __gconv_get_cache; __gconv_get_modules_db; + + # function used by the gconv modules + __gconv_transliterate; } } diff --git a/iconv/gconv.h b/iconv/gconv.h index 108dccbb46..7d59bb06c7 100644 --- a/iconv/gconv.h +++ b/iconv/gconv.h @@ -56,7 +56,8 @@ enum { __GCONV_IS_LAST = 0x0001, __GCONV_IGNORE_ERRORS = 0x0002, - __GCONV_SWAP = 0x0004 + __GCONV_SWAP = 0x0004, + __GCONV_TRANSLIT = 0x0008 }; @@ -64,7 +65,6 @@ enum struct __gconv_step; struct __gconv_step_data; struct __gconv_loaded_object; -struct __gconv_trans_data; /* Type of a conversion function. */ @@ -80,38 +80,6 @@ typedef int (*__gconv_init_fct) (struct __gconv_step *); typedef void (*__gconv_end_fct) (struct __gconv_step *); -/* Type of a transliteration/transscription function. */ -typedef int (*__gconv_trans_fct) (struct __gconv_step *, - struct __gconv_step_data *, void *, - const unsigned char *, - const unsigned char **, - const unsigned char *, unsigned char **, - size_t *); - -/* Function to call to provide transliteration module with context. */ -typedef int (*__gconv_trans_context_fct) (void *, const unsigned char *, - const unsigned char *, - unsigned char *, unsigned char *); - -/* Function to query module about supported encoded character sets. */ -typedef int (*__gconv_trans_query_fct) (const char *, const char ***, - size_t *); - -/* Constructor and destructor for local data for transliteration. */ -typedef int (*__gconv_trans_init_fct) (void **, const char *); -typedef void (*__gconv_trans_end_fct) (void *); - -struct __gconv_trans_data -{ - /* Transliteration/Transscription function. */ - __gconv_trans_fct __trans_fct; - __gconv_trans_context_fct __trans_context_fct; - __gconv_trans_end_fct __trans_end_fct; - void *__data; - struct __gconv_trans_data *__next; -}; - - /* Description of a conversion step. */ struct __gconv_step { @@ -163,9 +131,6 @@ struct __gconv_step_data __mbstate_t *__statep; __mbstate_t __state; /* This element must not be used directly by any module; always use STATEP! */ - - /* Transliteration information. */ - struct __gconv_trans_data *__trans; }; @@ -177,4 +142,13 @@ typedef struct __gconv_info __extension__ struct __gconv_step_data __data __flexarr; } *__gconv_t; +/* Transliteration using the locale's data. */ +extern int __gconv_transliterate (struct __gconv_step *step, + struct __gconv_step_data *step_data, + const unsigned char *inbufstart, + const unsigned char **inbufp, + const unsigned char *inbufend, + unsigned char **outbufstart, + size_t *irreversible); + #endif /* gconv.h */ diff --git a/iconv/gconv_close.c b/iconv/gconv_close.c index 81f0e0b319..f6394af6ca 100644 --- a/iconv/gconv_close.c +++ b/iconv/gconv_close.c @@ -37,20 +37,6 @@ __gconv_close (__gconv_t cd) drunp = cd->__data; do { - struct __gconv_trans_data *transp; - - transp = drunp->__trans; - while (transp != NULL) - { - struct __gconv_trans_data *curp = transp; - transp = transp->__next; - - if (__glibc_unlikely (curp->__trans_end_fct != NULL)) - curp->__trans_end_fct (curp->__data); - - free (curp); - } - if (!(drunp->__flags & __GCONV_IS_LAST) && drunp->__outbuf != NULL) free (drunp->__outbuf); } diff --git a/iconv/gconv_int.h b/iconv/gconv_int.h index ace076b88f..13b0e99915 100644 --- a/iconv/gconv_int.h +++ b/iconv/gconv_int.h @@ -92,21 +92,6 @@ struct gconv_module }; -/* Internal data structure to represent transliteration module. */ -struct trans_struct -{ - const char *name; - struct trans_struct *next; - - const char **csnames; - size_t ncsnames; - __gconv_trans_fct trans_fct; - __gconv_trans_context_fct trans_context_fct; - __gconv_trans_init_fct trans_init_fct; - __gconv_trans_end_fct trans_end_fct; -}; - - /* Flags for `gconv_open'. */ enum { @@ -258,20 +243,7 @@ extern void __gconv_get_builtin_trans (const char *name, struct __gconv_step *step) internal_function; -/* Try to load transliteration step module. */ -extern int __gconv_translit_find (struct trans_struct *trans) - internal_function; - -/* Transliteration using the locale's data. */ -extern int __gconv_transliterate (struct __gconv_step *step, - struct __gconv_step_data *step_data, - void *trans_data, - const unsigned char *inbufstart, - const unsigned char **inbufp, - const unsigned char *inbufend, - unsigned char **outbufstart, - size_t *irreversible) attribute_hidden; - +libc_hidden_proto (__gconv_transliterate) /* If NAME is an codeset alias expand it. */ extern int __gconv_compare_alias (const char *name1, const char *name2) diff --git a/iconv/gconv_open.c b/iconv/gconv_open.c index bfbe22bff6..615f33dfd2 100644 --- a/iconv/gconv_open.c +++ b/iconv/gconv_open.c @@ -39,7 +39,7 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, int conv_flags = 0; const char *errhand; const char *ignore; - struct trans_struct *trans = NULL; + bool translit = false; /* Find out whether any error handling method is specified. */ errhand = strchr (toset, '/'); @@ -66,72 +66,10 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, while (tok != NULL) { if (__strcasecmp_l (tok, "TRANSLIT", _nl_C_locobj_ptr) == 0) - { - /* It's the builtin transliteration handling. We only - support it for working on the internal encoding. */ - static const char *const internal_trans_names[1] - = { "INTERNAL" }; - struct trans_struct *lastp = NULL; - struct trans_struct *runp; - - for (runp = trans; runp != NULL; runp = runp->next) - if (runp->trans_fct == __gconv_transliterate) - break; - else - lastp = runp; - - if (runp == NULL) - { - struct trans_struct *newp; - - newp = (struct trans_struct *) alloca (sizeof (*newp)); - memset (newp, '\0', sizeof (*newp)); - - /* We leave the `name' field zero to signal that - this is an internal transliteration step. */ - newp->csnames = (const char **) internal_trans_names; - newp->ncsnames = 1; - newp->trans_fct = __gconv_transliterate; - - if (lastp == NULL) - trans = newp; - else - lastp->next = newp; - } - } + translit = true; else if (__strcasecmp_l (tok, "IGNORE", _nl_C_locobj_ptr) == 0) /* Set the flag to ignore all errors. */ conv_flags |= __GCONV_IGNORE_ERRORS; - else - { - /* `tok' is possibly a module name. We'll see later - whether we can find it. But first see that we do - not already a module of this name. */ - struct trans_struct *lastp = NULL; - struct trans_struct *runp; - - for (runp = trans; runp != NULL; runp = runp->next) - if (runp->name != NULL - && __strcasecmp_l (tok, runp->name, - _nl_C_locobj_ptr) == 0) - break; - else - lastp = runp; - - if (runp == NULL) - { - struct trans_struct *newp; - - newp = (struct trans_struct *) alloca (sizeof (*newp)); - memset (newp, '\0', sizeof (*newp)); - newp->name = tok; - - if (lastp == NULL) - trans = newp; - else - lastp->next = newp; - } - } tok = __strtok_r (NULL, ",", &ptr); } @@ -172,25 +110,6 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, res = __gconv_find_transform (toset, fromset, &steps, &nsteps, flags); if (res == __GCONV_OK) { - /* Find the modules. */ - struct trans_struct *lastp = NULL; - struct trans_struct *runp; - - for (runp = trans; runp != NULL; runp = runp->next) - { - if (runp->name == NULL - || __builtin_expect (__gconv_translit_find (runp), 0) == 0) - lastp = runp; - else - { - /* This means we haven't found the module. Remove it. */ - if (lastp == NULL) - trans = runp->next; - else - lastp->next = runp->next; - } - } - /* Allocate room for handle. */ result = (__gconv_t) malloc (sizeof (struct __gconv_info) + (nsteps @@ -199,8 +118,6 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, res = __GCONV_NOMEM; else { - size_t n; - /* Remember the list of steps. */ result->__steps = steps; result->__nsteps = nsteps; @@ -228,47 +145,12 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, /* We use the `mbstate_t' member in DATA. */ result->__data[cnt].__statep = &result->__data[cnt].__state; - /* Now see whether we can use any of the transliteration - modules for this step. */ - for (runp = trans; runp != NULL; runp = runp->next) - for (n = 0; n < runp->ncsnames; ++n) - if (__strcasecmp_l (steps[cnt].__from_name, - runp->csnames[n], _nl_C_locobj_ptr) == 0) - { - void *data = NULL; - - /* Match! Now try the initializer. */ - if (runp->trans_init_fct == NULL - || (runp->trans_init_fct (&data, - steps[cnt].__to_name) - == __GCONV_OK)) - { - /* Append at the end of the list. */ - struct __gconv_trans_data *newp; - struct __gconv_trans_data **lastp; - - newp = (struct __gconv_trans_data *) - malloc (sizeof (struct __gconv_trans_data)); - if (newp == NULL) - { - res = __GCONV_NOMEM; - goto bail; - } - - newp->__trans_fct = runp->trans_fct; - newp->__trans_context_fct = runp->trans_context_fct; - newp->__trans_end_fct = runp->trans_end_fct; - newp->__data = data; - newp->__next = NULL; - - lastp = &result->__data[cnt].__trans; - while (*lastp != NULL) - lastp = &(*lastp)->__next; - - *lastp = newp; - } - break; - } + /* The builtin transliteration handling only + supports the internal encoding. */ + if (translit + && __strcasecmp_l (steps[cnt].__from_name, + "INTERNAL", _nl_C_locobj_ptr) == 0) + conv_flags |= __GCONV_TRANSLIT; /* If this is the last step we must not allocate an output buffer. */ @@ -309,23 +191,7 @@ __gconv_open (const char *toset, const char *fromset, __gconv_t *handle, if (result != NULL) { while (cnt-- > 0) - { - struct __gconv_trans_data *transp; - - transp = result->__data[cnt].__trans; - while (transp != NULL) - { - struct __gconv_trans_data *curp = transp; - transp = transp->__next; - - if (__glibc_unlikely (curp->__trans_end_fct != NULL)) - curp->__trans_end_fct (curp->__data); - - free (curp); - } - - free (result->__data[cnt].__outbuf); - } + free (result->__data[cnt].__outbuf); free (result); result = NULL; diff --git a/iconv/gconv_trans.c b/iconv/gconv_trans.c index e0835fc666..65b5539307 100644 --- a/iconv/gconv_trans.c +++ b/iconv/gconv_trans.c @@ -32,7 +32,6 @@ int __gconv_transliterate (struct __gconv_step *step, struct __gconv_step_data *step_data, - void *trans_data __attribute__ ((unused)), const unsigned char *inbufstart, const unsigned char **inbufp, const unsigned char *inbufend, @@ -237,13 +236,4 @@ __gconv_transliterate (struct __gconv_step *step, /* Haven't found a match. */ return __GCONV_ILLEGAL_INPUT; } - -int -internal_function -__gconv_translit_find (struct trans_struct *trans) -{ - /* Transliteration module loading has been removed because it never - worked as intended and suffered from a security vulnerability. - Consequently, this function always fails. */ - return 1; -} +libc_hidden_def (__gconv_transliterate) diff --git a/iconv/loop.c b/iconv/loop.c index a480c0cd40..f4430ed968 100644 --- a/iconv/loop.c +++ b/iconv/loop.c @@ -213,8 +213,6 @@ points. */ #define STANDARD_TO_LOOP_ERR_HANDLER(Incr) \ { \ - struct __gconv_trans_data *trans; \ - \ result = __GCONV_ILLEGAL_INPUT; \ \ if (irreversible == NULL) \ @@ -227,14 +225,10 @@ UPDATE_PARAMS; \ \ /* First try the transliteration methods. */ \ - for (trans = step_data->__trans; trans != NULL; trans = trans->__next) \ - { \ - result = DL_CALL_FCT (trans->__trans_fct, \ - (step, step_data, trans->__data, *inptrp, \ - &inptr, inend, &outptr, irreversible)); \ - if (result != __GCONV_ILLEGAL_INPUT) \ - break; \ - } \ + if ((step_data->__flags & __GCONV_TRANSLIT) != 0) \ + result = __gconv_transliterate \ + (step, step_data, *inptrp, \ + &inptr, inend, &outptr, irreversible); \ \ REINIT_PARAMS; \ \ diff --git a/iconv/skeleton.c b/iconv/skeleton.c index 73dc1860a9..acd60e2f88 100644 --- a/iconv/skeleton.c +++ b/iconv/skeleton.c @@ -501,8 +501,9 @@ FUNCTION_NAME (struct __gconv_step *step, struct __gconv_step_data *data, } else { - /* We preserve the initial values of the pointer variables. */ - const unsigned char *inptr = *inptrp; + /* We preserve the initial values of the pointer variables, + but only some conversion modules need it. */ + const unsigned char *inptr __attribute__ ((__unused__)) = *inptrp; unsigned char *outbuf = (__builtin_expect (outbufstart == NULL, 1) ? data->__outbuf : *outbufstart); unsigned char *outend = data->__outbufend; @@ -592,8 +593,6 @@ FUNCTION_NAME (struct __gconv_step *step, struct __gconv_step_data *data, while (1) { - struct __gconv_trans_data *trans; - /* Remember the start value for this round. */ inptr = *inptrp; /* The outbuf buffer is empty. */ @@ -640,13 +639,6 @@ FUNCTION_NAME (struct __gconv_step *step, struct __gconv_step_data *data, return status; } - /* Give the transliteration module the chance to store the - original text and the result in case it needs a context. */ - for (trans = data->__trans; trans != NULL; trans = trans->__next) - if (trans->__trans_context_fct != NULL) - DL_CALL_FCT (trans->__trans_context_fct, - (trans->__data, inptr, *inptrp, outstart, outbuf)); - /* We finished one use of the loops. */ ++data->__invocation_counter; diff --git a/libio/fileops.c b/libio/fileops.c index 204cfeaa35..e0d7b76eda 100644 --- a/libio/fileops.c +++ b/libio/fileops.c @@ -403,21 +403,16 @@ _IO_new_file_fopen (fp, filename, mode, is32not64) cc->__cd_in.__cd.__data[0].__flags = __GCONV_IS_LAST; cc->__cd_in.__cd.__data[0].__statep = &result->_wide_data->_IO_state; - /* XXX For now no transliteration. */ - cc->__cd_in.__cd.__data[0].__trans = NULL; - cc->__cd_out.__cd.__nsteps = fcts.tomb_nsteps; cc->__cd_out.__cd.__steps = fcts.tomb; cc->__cd_out.__cd.__data[0].__invocation_counter = 0; cc->__cd_out.__cd.__data[0].__internal_use = 1; - cc->__cd_out.__cd.__data[0].__flags = __GCONV_IS_LAST; + cc->__cd_out.__cd.__data[0].__flags + = __GCONV_IS_LAST | __GCONV_TRANSLIT; cc->__cd_out.__cd.__data[0].__statep = &result->_wide_data->_IO_state; - /* And now the transliteration. */ - cc->__cd_out.__cd.__data[0].__trans = &__libio_translit; - /* From now on use the wide character callback functions. */ ((struct _IO_FILE_plus *) fp)->vtable = fp->_wide_data->_wide_vtable; diff --git a/libio/iofwide.c b/libio/iofwide.c index 64187e401b..ecf5819856 100644 --- a/libio/iofwide.c +++ b/libio/iofwide.c @@ -81,14 +81,6 @@ const struct _IO_codecvt __libio_codecvt = }; -#ifdef _LIBC -const struct __gconv_trans_data __libio_translit attribute_hidden = -{ - .__trans_fct = __gconv_transliterate -}; -#endif - - /* Return orientation of stream. If mode is nonzero try to change the orientation first. */ #undef _IO_fwide @@ -146,20 +138,14 @@ _IO_fwide (fp, mode) cc->__cd_in.__cd.__data[0].__flags = __GCONV_IS_LAST; cc->__cd_in.__cd.__data[0].__statep = &fp->_wide_data->_IO_state; - /* XXX For now no transliteration. */ - cc->__cd_in.__cd.__data[0].__trans = NULL; - cc->__cd_out.__cd.__nsteps = fcts.tomb_nsteps; cc->__cd_out.__cd.__steps = fcts.tomb; cc->__cd_out.__cd.__data[0].__invocation_counter = 0; cc->__cd_out.__cd.__data[0].__internal_use = 1; - cc->__cd_out.__cd.__data[0].__flags = __GCONV_IS_LAST; + cc->__cd_out.__cd.__data[0].__flags + = __GCONV_IS_LAST | __GCONV_TRANSLIT; cc->__cd_out.__cd.__data[0].__statep = &fp->_wide_data->_IO_state; - - /* And now the transliteration. */ - cc->__cd_out.__cd.__data[0].__trans - = (struct __gconv_trans_data *) &__libio_translit; } #else # ifdef _GLIBCPP_USE_WCHAR_T diff --git a/wcsmbs/btowc.c b/wcsmbs/btowc.c index 289736ff92..aafb39209f 100644 --- a/wcsmbs/btowc.c +++ b/wcsmbs/btowc.c @@ -75,7 +75,6 @@ __btowc (c) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = &data.__state; - data.__trans = NULL; /* Make sure we start in the initial state. */ memset (&data.__state, '\0', sizeof (mbstate_t)); diff --git a/wcsmbs/mbrtoc16.c b/wcsmbs/mbrtoc16.c index 643aaf585c..69105bae55 100644 --- a/wcsmbs/mbrtoc16.c +++ b/wcsmbs/mbrtoc16.c @@ -67,7 +67,6 @@ mbrtoc16 (char16_t *pc16, const char *s, size_t n, mbstate_t *ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps; - data.__trans = NULL; /* A first special case is if S is NULL. This means put PS in the initial state. */ diff --git a/wcsmbs/mbrtowc.c b/wcsmbs/mbrtowc.c index c57217add5..8070bd8a3d 100644 --- a/wcsmbs/mbrtowc.c +++ b/wcsmbs/mbrtowc.c @@ -49,7 +49,6 @@ __mbrtowc (wchar_t *pwc, const char *s, size_t n, mbstate_t *ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps ?: &state; - data.__trans = NULL; /* A first special case is if S is NULL. This means put PS in the initial state. */ diff --git a/wcsmbs/mbsnrtowcs.c b/wcsmbs/mbsnrtowcs.c index f91e580e0d..e611ddee76 100644 --- a/wcsmbs/mbsnrtowcs.c +++ b/wcsmbs/mbsnrtowcs.c @@ -58,7 +58,6 @@ __mbsnrtowcs (dst, src, nmc, len, ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps ?: &state; - data.__trans = NULL; if (nmc == 0) return 0; diff --git a/wcsmbs/mbsrtowcs_l.c b/wcsmbs/mbsrtowcs_l.c index 08ff3c9ced..5e10a7efd4 100644 --- a/wcsmbs/mbsrtowcs_l.c +++ b/wcsmbs/mbsrtowcs_l.c @@ -56,7 +56,6 @@ __mbsrtowcs_l (dst, src, len, ps, l) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps; - data.__trans = NULL; /* Get the conversion functions. */ fcts = get_gconv_fcts (l->__locales[LC_CTYPE]); diff --git a/wcsmbs/wcrtomb.c b/wcsmbs/wcrtomb.c index be93877ccc..67c68d36a0 100644 --- a/wcsmbs/wcrtomb.c +++ b/wcsmbs/wcrtomb.c @@ -49,7 +49,6 @@ __wcrtomb (char *s, wchar_t wc, mbstate_t *ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps ?: &state; - data.__trans = NULL; /* A first special case is if S is NULL. This means put PS in the initial state. */ diff --git a/wcsmbs/wcsnrtombs.c b/wcsmbs/wcsnrtombs.c index 6fe718dd26..015d08c170 100644 --- a/wcsmbs/wcsnrtombs.c +++ b/wcsmbs/wcsnrtombs.c @@ -56,7 +56,6 @@ __wcsnrtombs (dst, src, nwc, len, ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps ?: &state; - data.__trans = NULL; if (nwc == 0) return 0; diff --git a/wcsmbs/wcsrtombs.c b/wcsmbs/wcsrtombs.c index 24e249c6eb..988b468e85 100644 --- a/wcsmbs/wcsrtombs.c +++ b/wcsmbs/wcsrtombs.c @@ -52,7 +52,6 @@ __wcsrtombs (dst, src, len, ps) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = ps ?: &state; - data.__trans = NULL; /* Get the conversion functions. */ fcts = get_gconv_fcts (_NL_CURRENT_DATA (LC_CTYPE)); diff --git a/wcsmbs/wctob.c b/wcsmbs/wctob.c index 24c33cec06..8e65738685 100644 --- a/wcsmbs/wctob.c +++ b/wcsmbs/wctob.c @@ -53,7 +53,6 @@ wctob (c) data.__internal_use = 1; data.__flags = __GCONV_IS_LAST; data.__statep = &data.__state; - data.__trans = NULL; /* Make sure we start in the initial state. */ memset (&data.__state, '\0', sizeof (mbstate_t)); |