diff options
-rw-r--r-- | filter/ChangeLog | 13 | ||||
-rw-r--r-- | filter/filter-code.c | 2 | ||||
-rw-r--r-- | filter/filter-folder.c | 15 | ||||
-rw-r--r-- | filter/filter-input.c | 4 | ||||
-rw-r--r-- | filter/filter-option.c | 4 |
5 files changed, 29 insertions, 9 deletions
diff --git a/filter/ChangeLog b/filter/ChangeLog index cbd2332adf..b8148bbf48 100644 --- a/filter/ChangeLog +++ b/filter/ChangeLog @@ -1,5 +1,18 @@ 2001-08-08 Radek Doulik <rodo@ximian.com> + * filter-folder.c (button_clicked): don't mix g_free and xmlAlloc + (xml_decode): don't mix g_free and xmlAlloc + + * filter-code.c (filter_code_init): use xmlStrdup instead of + g_strdup + + * filter-input.c (filter_input_new_type_name): use xmlStrdup + instead of g_strdup + (filter_input_finalise): don't mix g_free and xmlAlloc + + * filter-option.c (clone): use xmlStrdup instead of g_strdup + (free_option): don't mix g_free and xmlAlloc + * filter-source.c (clone): use xmlStrdup rather than g_strdup * filter-element.c (filter_element_finalise): don't mix g_free and diff --git a/filter/filter-code.c b/filter/filter-code.c index ecdccc4b97..9ae83468f9 100644 --- a/filter/filter-code.c +++ b/filter/filter-code.c @@ -75,7 +75,7 @@ filter_code_class_init (FilterCodeClass *class) static void filter_code_init (FilterCode *o) { - ((FilterInput *)o)->type = g_strdup("code"); + ((FilterInput *)o)->type = xmlStrdup("code"); } static void diff --git a/filter/filter-folder.c b/filter/filter-folder.c index 4b43997c2b..c79059178e 100644 --- a/filter/filter-folder.c +++ b/filter/filter-folder.c @@ -193,10 +193,17 @@ xml_decode (FilterElement *fe, xmlNodePtr node) n = node->childs; while (n) { if (!strcmp (n->name, "folder")) { - xmlFree (ff->name); - xmlFree (ff->uri); - ff->name = xmlGetProp (n, "name"); - ff->uri = xmlGetProp (n, "uri"); + char *uri, *name; + + name = xmlGetProp (n, "name"); + g_free (ff->name); + ff->name = g_strdup (name); + xmlFree (name); + + uri = xmlGetProp (n, "uri"); + g_free (ff->uri); + ff->uri = g_strdup (uri); + xmlFree (uri); break; } n = n->next; diff --git a/filter/filter-input.c b/filter/filter-input.c index e25d3e40ee..8bf430993b 100644 --- a/filter/filter-input.c +++ b/filter/filter-input.c @@ -121,7 +121,7 @@ filter_input_finalise (GtkObject *obj) { FilterInput *o = (FilterInput *)obj; - g_free(o->type); + xmlFree (o->type); g_list_foreach(o->values, (GFunc)g_free, NULL); g_list_free(o->values); @@ -148,7 +148,7 @@ FilterInput * filter_input_new_type_name (const char *type) { FilterInput *o = filter_input_new (); - o->type = g_strdup (type); + o->type = xmlStrdup (type); d(printf("new type %s = %p\n", type, o)); return o; diff --git a/filter/filter-option.c b/filter/filter-option.c index 9648bc36a0..a517fd71df 100644 --- a/filter/filter-option.c +++ b/filter/filter-option.c @@ -116,7 +116,7 @@ static void free_option(struct _filter_option *o, void *data) { g_free(o->title); - g_free(o->value); + xmlFree (o->value); g_free(o->code); g_free(o); } @@ -343,7 +343,7 @@ clone (FilterElement *fe) fn = g_malloc (sizeof (*fn)); d(printf (" option %s\n", op->title)); fn->title = g_strdup (op->title); - fn->value = g_strdup (op->value); + fn->value = xmlStrdup (op->value); if (op->code) fn->code = g_strdup (op->code); else |