aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-11-01 14:04:15 +0100
committerMax Kellermann <max@duempel.org>2008-11-01 14:04:15 +0100
commitb996108675896e6b4be08f78e32f28bc6d454ad9 (patch)
treeede9f7297786a4da8c780c8c1118d3efcc756d85 /src
parentaa33422de680989c5b7c4a91684886fd42e382cc (diff)
downloadmpd-b996108675896e6b4be08f78e32f28bc6d454ad9.tar.gz
mpd-b996108675896e6b4be08f78e32f28bc6d454ad9.tar.xz
mpd-b996108675896e6b4be08f78e32f28bc6d454ad9.zip
tag: don't return const string from fix_utf8()
Return NULL instead of the input value if there is nothing to fix. This way, the caller doesn't have to use the xfree() hack.
Diffstat (limited to 'src')
-rw-r--r--src/tag.c26
1 files changed, 13 insertions, 13 deletions
diff --git a/src/tag.c b/src/tag.c
index be4cd1887..cb0508aed 100644
--- a/src/tag.c
+++ b/src/tag.c
@@ -343,22 +343,24 @@ int tag_equal(const struct tag *tag1, const struct tag *tag2)
return 1;
}
-static inline const char *fix_utf8(const char *str, size_t *length_r) {
- const char *temp;
+static char *
+fix_utf8(const char *str, size_t *length_r)
+{
+ char *temp;
GError *error = NULL;
gsize written;
assert(str != NULL);
if (g_utf8_validate(str, *length_r, NULL))
- return str;
+ return NULL;
DEBUG("not valid utf8 in tag: %s\n",str);
temp = g_convert(str, *length_r, "utf-8", "iso-8859-1",
NULL, &written, &error);
if (temp == NULL) {
g_error_free(error);
- return str;
+ return NULL;
}
*length_r = written;
@@ -401,18 +403,17 @@ static void appendToTagItems(struct tag *tag, enum tag_type type,
const char *value, size_t len)
{
unsigned int i = tag->numOfItems;
+ char *duplicated;
const char *p;
- p = fix_utf8(value, &len);
+ p = duplicated = fix_utf8(value, &len);
+ if (p == NULL)
+ p = value;
if (memchr(p, '\n', len) != NULL) {
- char *duplicated = xmalloc(len + 1);
- memcpy(duplicated, p, len);
- duplicated[len] = '\0';
- if (p != value)
- xfree(p);
+ if (duplicated == NULL)
+ p = duplicated = g_strndup(p, len);
stripReturnChar(duplicated);
- p = duplicated;
}
tag->numOfItems++;
@@ -433,8 +434,7 @@ static void appendToTagItems(struct tag *tag, enum tag_type type,
tag->items[i] = tag_pool_get_item(type, p, len);
pthread_mutex_unlock(&tag_pool_lock);
- if (p != value)
- xfree(p);
+ g_free(duplicated);
}
void tag_add_item_n(struct tag *tag, enum tag_type itemType,