Fix memory leaks.
[kopensolaris-gnu/glibc.git] / locale / setlocale.c
index 368027f..3e63b25 100644 (file)
@@ -49,6 +49,9 @@ static struct locale_data * *const _nl_current[] =
     [category] = &_nl_current_##category,
 #include "categories.def"
 #undef DEFINE_CATEGORY
+    /* We need this additional element to simplify the code.  It must
+       simply be != NULL.  */
+    [LC_ALL] = (struct locale_data **) ~0ul
   };
 
 /* Array indexed by category of pointers to _nl_C_CATEGORY slots.
@@ -201,8 +204,7 @@ new_composite_name (int category, const char *newnames[LC_ALL])
 static inline void
 setname (int category, const char *name)
 {
-  if (_nl_current[category] == NULL
-      && _nl_current_names[category] != _nl_C_name)
+  if (_nl_current_names[category] != _nl_C_name)
     free ((void *) _nl_current_names[category]);
 
   _nl_current_names[category] = name;
@@ -314,28 +316,19 @@ setlocale (int category, const char *locale)
 
       /* Load the new data for each category.  */
       while (category-- > 0)
-       /* Only actually load the data if anything will use it.  */
-       if (_nl_current[category] != NULL)
-         {
-           newdata[category] = _nl_find_locale (locale_path, locale_path_len,
-                                                category,
-                                                &newnames[category]);
-
-           if (newdata[category] == NULL)
-             break;
-
-           /* We must not simply free a global locale since we have
-              no control over the usage.  So we mark it as
-              un-deletable.  */
+       {
+         newdata[category] = _nl_find_locale (locale_path, locale_path_len,
+                                              category,
+                                              &newnames[category]);
+
+         if (newdata[category] == NULL)
+           break;
+
+         /* We must not simply free a global locale since we have no
+            control over the usage.  So we mark it as un-deletable.  */
+         if (newdata[category]->usage_count != MAX_USAGE_COUNT)
            newdata[category]->usage_count = MAX_USAGE_COUNT;
-         }
-       else
-         {
-           /* The data is never used; just change the name.  */
-           newnames[category] = clever_copy (newnames[category]);
-           if (newnames[category] == NULL)
-             break;
-         }
+       }
 
       /* Create new composite name.  */
       if (category >= 0
@@ -357,6 +350,9 @@ setlocale (int category, const char *locale)
       /* Critical section left.  */
       __libc_lock_unlock (__libc_setlocale_lock);
 
+      /* Free the resources (the locale path variable.  */
+      free (locale_path);
+
       return composite;
     }
   else
@@ -377,7 +373,8 @@ setlocale (int category, const char *locale)
 
          /* We must not simply free a global locale since we have no
             control over the usage.  So we mark it as un-deletable.  */
-         newdata->usage_count = MAX_USAGE_COUNT;
+         if (newdata->usage_count != MAX_USAGE_COUNT)
+           newdata->usage_count = MAX_USAGE_COUNT;
        }
 
       /* Create new composite name.  */
@@ -400,6 +397,9 @@ setlocale (int category, const char *locale)
       /* Critical section left.  */
       __libc_lock_unlock (__libc_setlocale_lock);
 
+      /* Free the resources (the locale path variable.  */
+      free (locale_path);
+
       return (char *) newname;
     }
 }