about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog3
-rw-r--r--locale/programs/ld-numeric.c15
2 files changed, 7 insertions, 11 deletions
diff --git a/ChangeLog b/ChangeLog
index 712002e665..4d88955fe4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2000-01-29  Ulrich Drepper  <drepper@redhat.com>
 
+	* locale/programs/ld-numeric.c (numeric_output): Increment cnt in
+	correct place to avoid uninitialized memory passed to writev.
+
 	* locale/programs/ld-monetary.c (STR_ELEM_WC): Fix test for too
 	long string in definition.Only enable returning wide character
 	strings if really needed.
diff --git a/locale/programs/ld-numeric.c b/locale/programs/ld-numeric.c
index 37951c8442..017c41fdc2 100644
--- a/locale/programs/ld-numeric.c
+++ b/locale/programs/ld-numeric.c
@@ -104,15 +104,6 @@ numeric_finish (struct localedef_t *locale, struct charmap_t *charmap)
 	}
     }
 
-#define TEST_ELEM(cat, default)						      \
-  if (numeric->cat == NULL && ! be_quiet && ! nothing)			      \
-    error (0, 0, _("%s: field `%s' not defined"), "LC_NUMERIC", #cat);	      \
-  if (numeric->cat##_wc == L'\0')					      \
-    numeric->cat##_wc = default
-
-  TEST_ELEM (decimal_point, L'.');
-  TEST_ELEM (thousands_sep, L'\0');
-
   /* The decimal point must not be empty.  This is not said explicitly
      in POSIX but ANSI C (ISO/IEC 9899) says in 4.4.2.1 it has to be
      != "".  */
@@ -129,6 +120,8 @@ numeric_finish (struct localedef_t *locale, struct charmap_t *charmap)
 %s: value for field `%s' must not be the empty string"),
 	     "LC_NUMERIC", "decimal_point");
     }
+  if (numeric->decimal_point_wc == L'\0')
+    numeric->decimal_point_wc = L'.';
 
   if (numeric->grouping_len == 0 && ! be_quiet && ! nothing)
     error (0, 0, _("%s: field `%s' not defined"), "LC_NUMERIC", "grouping");
@@ -168,6 +161,7 @@ numeric_output (struct localedef_t *locale, struct charmap_t *charmap,
   idx[cnt - 2] = idx[cnt - 3] + iov[cnt - 1].iov_len;
   iov[cnt].iov_base = numeric->grouping;
   iov[cnt].iov_len = numeric->grouping_len;
+  ++cnt;
 
   idx[cnt - 2] = iov[0].iov_len + iov[1].iov_len;
   iov[cnt].iov_base = (void *) &numeric->decimal_point_wc;
@@ -176,8 +170,7 @@ numeric_output (struct localedef_t *locale, struct charmap_t *charmap,
 
   idx[cnt - 2] = idx[cnt - 3] + iov[cnt - 1].iov_len;
   iov[cnt].iov_base = (void *) &numeric->thousands_sep_wc;
-  iov[cnt].iov_len = sizeof (uint32_t);;
-  ++cnt;
+  iov[cnt].iov_len = sizeof (uint32_t);
 
   assert (cnt + 1 == 2 + _NL_ITEM_INDEX (_NL_NUM_LC_NUMERIC));