Mercurial > libavformat.hg
changeset 5715:ead2df871971 libavformat
Read ASF metadata as proper UTF-16 and spit it out as proper UTF-8 in our
metadata system.
Patch by Anton Khirnov <wyskas gmail com>.
| author | rbultje |
|---|---|
| date | Wed, 24 Feb 2010 18:19:54 +0000 |
| parents | 081abaf38c51 |
| children | 0dcb8b662232 |
| files | asfdec.c |
| diffstat | 1 files changed, 10 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/asfdec.c Wed Feb 24 18:17:32 2010 +0000 +++ b/asfdec.c Wed Feb 24 18:19:54 2010 +0000 @@ -122,9 +122,12 @@ static void get_str16_nolen(ByteIOContext *pb, int len, char *buf, int buf_size) { char* q = buf; - for (; len > 1; len -= 2) { + while (len > 1) { uint8_t tmp; - PUT_UTF8(get_le16(pb), tmp, if (q - buf < buf_size - 1) *q++ = tmp;) + uint32_t ch; + + GET_UTF16(ch, (len -= 2) >= 0 ? get_le16(pb) : 0, break;) + PUT_UTF8(ch, tmp, if (q - buf < buf_size - 1) *q++ = tmp;) } if (len > 0) url_fskip(pb, len); @@ -154,15 +157,15 @@ { char *value; - if ((unsigned)len >= UINT_MAX) + if ((unsigned)len >= (UINT_MAX - 1)/2) return; - value = av_malloc(len+1); + value = av_malloc(2*len+1); if (!value) return; if (type == 0) { // UTF16-LE - get_str16_nolen(s->pb, len, value, len); + get_str16_nolen(s->pb, len, value, 2*len + 1); } else if (type > 1 && type <= 5) { // boolean or DWORD or QWORD or WORD uint64_t num = get_value(s->pb, type); snprintf(value, len, "%"PRIu64, num); @@ -174,7 +177,8 @@ } if (!strncmp(key, "WM/", 3)) key += 3; - av_metadata_set2(&s->metadata, key, value, AV_METADATA_DONT_STRDUP_VAL); + av_metadata_set2(&s->metadata, key, value, 0); + av_freep(&value); } static int asf_read_header(AVFormatContext *s, AVFormatParameters *ap)