X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=aac_afh.c;h=c4301a2f178257b19d56b3cf297da3e1eaff337c;hb=4ce5b6570ed83ef2ab5296f369f147593507286b;hp=84aeaa85e6a07bf37033856587eb2f8600cedf01;hpb=a4d904cc19f2b088cc1c72d9f5ce1fcb998e2dcb;p=paraslash.git

diff --git a/aac_afh.c b/aac_afh.c
index 84aeaa85..c4301a2f 100644
--- a/aac_afh.c
+++ b/aac_afh.c
@@ -22,12 +22,11 @@ struct aac_afh_context {
 	const void *map;
 	size_t mapsize;
 	size_t fpos;
-	int32_t track;
 	struct mp4 *mp4;
 	struct mp4_callback cb;
 };
 
-static uint32_t aac_afh_read_cb(void *user_data, void *dest, uint32_t want)
+static ssize_t aac_afh_read_cb(void *user_data, void *dest, size_t want)
 {
 	struct aac_afh_context *c = user_data;
 	size_t have, rv;
@@ -35,35 +34,32 @@ static uint32_t aac_afh_read_cb(void *user_data, void *dest, uint32_t want)
 	if (want == 0 || c->fpos >= c->mapsize)
 		return 0;
 	have = c->mapsize - c->fpos;
-	rv = PARA_MIN(have, (size_t)want);
+	rv = PARA_MIN(have, want);
 	PARA_DEBUG_LOG("reading %zu bytes @%zu\n", rv, c->fpos);
 	memcpy(dest, c->map + c->fpos, rv);
 	c->fpos += rv;
 	return rv;
 }
 
-static uint32_t aac_afh_seek_cb(void *user_data, uint64_t pos)
+static off_t aac_afh_seek_cb(void *user_data, off_t offset, int whence)
 {
 	struct aac_afh_context *c = user_data;
-	c->fpos = pos;
-	return 0;
-}
-
-static int32_t aac_afh_get_track(struct mp4 *mp4)
-{
-	int32_t i, num_tracks = mp4_total_tracks(mp4);
 
-	assert(num_tracks >= 0);
-	for (i = 0; i < num_tracks; i++)
-		if (mp4_is_audio_track(mp4, i))
-			return i;
-	return -E_MP4_TRACK; /* no audio track */
+	if (whence == SEEK_SET)
+		c->fpos = offset;
+	else if (whence == SEEK_CUR)
+		c->fpos += offset;
+	else if (whence == SEEK_END)
+		c->fpos = c->mapsize + offset;
+	else
+		assert(false);
+	return c->fpos;
 }
 
 static int aac_afh_open(const void *map, size_t mapsize, void **afh_context)
 {
 	int ret;
-	struct aac_afh_context *c = para_malloc(sizeof(*c));
+	struct aac_afh_context *c = alloc(sizeof(*c));
 
 	c->map = map;
 	c->mapsize = mapsize;
@@ -72,18 +68,11 @@ static int aac_afh_open(const void *map, size_t mapsize, void **afh_context)
 	c->cb.seek = aac_afh_seek_cb;
 	c->cb.user_data = c;
 
-	ret = -E_MP4_OPEN;
-	c->mp4 = mp4_open_read(&c->cb);
-	if (!c->mp4)
-		goto free_ctx;
-	ret = aac_afh_get_track(c->mp4);
+	ret = mp4_open(&c->cb, &c->mp4);
 	if (ret < 0)
-		goto close_mp4;
-	c->track = ret;
+		goto free_ctx;
 	*afh_context = c;
 	return 0;
-close_mp4:
-	mp4_close(c->mp4);
 free_ctx:
 	free(c);
 	*afh_context = NULL;
@@ -101,28 +90,31 @@ static int aac_afh_get_chunk(uint32_t chunk_num, void *afh_context,
 		const char **buf, uint32_t *len)
 {
 	struct aac_afh_context *c = afh_context;
-	int32_t ss;
+	uint32_t ss;
 	size_t offset;
+	int ret;
 
-	assert(chunk_num <= INT_MAX);
-	mp4_set_sample_position(c->mp4, c->track, chunk_num);
+	ret = mp4_set_sample_position(c->mp4, chunk_num);
+	if (ret < 0)
+		return ret;
 	offset = c->fpos;
-	ss = mp4_get_sample_size(c->mp4, c->track, chunk_num);
-	if (ss <= 0)
-		return -E_MP4_BAD_SAMPLE;
-	assert(ss + offset <= c->mapsize);
+	ret = mp4_get_sample_size(c->mp4, chunk_num, &ss);
+	if (ret < 0)
+		return ret;
+	if (ss + offset > c->mapsize) /* file got truncated?! */
+		return -E_MP4_CORRUPT;
 	*buf = c->map + offset;
 	*len = ss;
 	return 1;
 }
 
-static void _aac_afh_get_taginfo(const struct mp4 *mp4, struct taginfo *tags)
+static void aac_afh_get_taginfo(const struct mp4 *mp4, struct taginfo *tags)
 {
-	tags->artist = mp4_meta_get_artist(mp4);
-	tags->title = mp4_meta_get_title(mp4);
-	tags->year = mp4_meta_get_date(mp4);
-	tags->album = mp4_meta_get_album(mp4);
-	tags->comment = mp4_meta_get_comment(mp4);
+	tags->artist = mp4_get_tag_value(mp4, "artist");
+	tags->title = mp4_get_tag_value(mp4, "title");
+	tags->year = mp4_get_tag_value(mp4, "date");
+	tags->album = mp4_get_tag_value(mp4, "album");
+	tags->comment = mp4_get_tag_value(mp4, "comment");
 }
 
 /*
@@ -132,7 +124,6 @@ static int aac_get_file_info(char *map, size_t numbytes, __a_unused int fd,
 		struct afh_info *afhi)
 {
 	int ret;
-	int32_t rv;
 	struct aac_afh_context *c;
 	uint64_t milliseconds;
 	const char *buf;
@@ -142,87 +133,90 @@ static int aac_get_file_info(char *map, size_t numbytes, __a_unused int fd,
 	if (ret < 0)
 		return ret;
 
-	ret = -E_MP4_BAD_SAMPLERATE;
-	rv = mp4_get_sample_rate(c->mp4, c->track);
-	if (rv <= 0)
-		goto close;
-	afhi->frequency = rv;
-
-	ret = -E_MP4_BAD_CHANNEL_COUNT;
-	rv = mp4_get_channel_count(c->mp4, c->track);
-	if (rv <= 0)
-		goto close;
-	afhi->channels = rv;
-
-	ret = -E_MP4_BAD_SAMPLE_COUNT;
-	rv = mp4_num_samples(c->mp4, c->track);
-	if (rv <= 0)
-		goto close;
-	afhi->chunks_total = rv;
+	afhi->frequency = mp4_get_sample_rate(c->mp4);
+	assert(afhi->frequency > 0);
+	afhi->channels = mp4_get_channel_count(c->mp4);
+	assert(afhi->channels > 0);
+	afhi->chunks_total = mp4_num_samples(c->mp4);
+	assert(afhi->chunks_total > 0);
+
 	afhi->max_chunk_size = 0;
 	for (n = 0; n < afhi->chunks_total; n++) {
-		if (aac_afh_get_chunk(n, c, &buf, &len) < 0)
-			break;
+		ret = aac_afh_get_chunk(n, c, &buf, &len);
+		if (ret < 0)
+			goto out;
 		afhi->max_chunk_size = PARA_MAX(afhi->max_chunk_size, len);
 	}
-	milliseconds = mp4_get_duration(c->mp4, c->track);
+	milliseconds = mp4_get_duration(c->mp4);
 	afhi->seconds_total = milliseconds / 1000;
 	ms2tv(milliseconds / afhi->chunks_total, &afhi->chunk_tv);
-	if (aac_afh_get_chunk(0, c, &buf, &len) >= 0)
-		numbytes -= buf - map;
+	if (aac_afh_get_chunk(0, c, &buf, &len) < 0)
+		goto out;
+	numbytes -= buf - map;
 	afhi->bitrate = 8 * numbytes / afhi->seconds_total / 1000;
-	_aac_afh_get_taginfo(c->mp4, &afhi->tags);
+	aac_afh_get_taginfo(c->mp4, &afhi->tags);
 	ret = 1;
-close:
+out:
 	aac_afh_close(c);
 	return ret;
 }
 
-static uint32_t aac_afh_meta_read_cb(void *user_data, void *dest, uint32_t want)
+static ssize_t aac_afh_meta_read_cb(void *user_data, void *dest, size_t want)
 {
 	int fd = *(int *)user_data;
 	return read(fd, dest, want);
 }
 
-static uint32_t aac_afh_meta_seek_cb(void *user_data, uint64_t pos)
+static off_t aac_afh_meta_seek_cb(void *user_data, off_t offset, int whence)
 {
 	int fd = *(int *)user_data;
-	return lseek(fd, pos, SEEK_SET);
+	off_t ret = lseek(fd, offset, whence);
+
+	assert(ret != (off_t)-1);
+	return ret;
 }
 
-static uint32_t aac_afh_meta_write_cb(void *user_data, void *dest, uint32_t want)
+static ssize_t aac_afh_meta_write_cb(void *user_data, void *dest, size_t count)
 {
 	int fd = *(int *)user_data;
-	return write(fd, dest, want);
+	return write(fd, dest, count);
 }
 
-static uint32_t aac_afh_meta_truncate_cb(void *user_data)
+static int aac_afh_meta_truncate_cb(void *user_data)
 {
 	int fd = *(int *)user_data;
 	off_t offset = lseek(fd, 0, SEEK_CUR);
 	return ftruncate(fd, offset);
 }
 
-static void replace_tag(struct mp4_tag *tag, const char *new_val, bool *found)
-{
-	free(tag->value);
-	tag->value = para_strdup(new_val);
-	*found = true;
-}
-
-static void add_tag(struct mp4_metadata *md, const char *item, const char *value)
+static void replace_or_add_tag(const char *item, const char *value,
+		struct mp4_metadata *meta)
 {
-	md->tags[md->count].item = para_strdup(item);
-	md->tags[md->count].value = para_strdup(value);
-	md->count++;
+	uint32_t n;
+	struct mp4_tag *t;
+
+	for (n = 0; n < meta->count; n++) {
+		t = meta->tags + n;
+		if (strcasecmp(t->item, item))
+			continue;
+		free(t->value);
+		t->value = para_strdup(value);
+		return;
+	}
+	/* item not found, add new tag */
+	meta->tags = para_realloc(meta->tags, (meta->count + 1)
+		* sizeof(struct mp4_tag));
+	t = meta->tags + meta->count;
+	t->item = para_strdup(item);
+	t->value = para_strdup(value);
+	meta->count++;
 }
 
 static int aac_afh_rewrite_tags(const char *map, size_t mapsize,
 		struct taginfo *tags, int fd, __a_unused const char *filename)
 {
-	int ret, i;
-	int32_t rv;
-	struct mp4_metadata metadata;
+	int ret;
+	struct mp4_metadata *metadata;
 	struct mp4 *mp4;
 	struct mp4_callback cb = {
 		.read = aac_afh_meta_read_cb,
@@ -231,59 +225,23 @@ static int aac_afh_rewrite_tags(const char *map, size_t mapsize,
 		.truncate = aac_afh_meta_truncate_cb,
 		.user_data = &fd
 	};
-	bool found_artist = false, found_title = false, found_album = false,
-		found_year = false, found_comment = false;
 
 	ret = write_all(fd, map, mapsize);
 	if (ret < 0)
 		return ret;
 	lseek(fd, 0, SEEK_SET);
 
-	mp4 = mp4_open_meta(&cb);
-	if (!mp4)
-		return -E_MP4_OPEN;
-
-	ret = -E_MP4_META_READ;
-	rv = mp4_meta_get_num_items(mp4);
-	if (rv < 0)
-		goto close;
-	metadata.count = rv;
-	PARA_NOTICE_LOG("%d metadata item(s) found\n", rv);
-
-	metadata.tags = para_malloc((metadata.count + 5) * sizeof(struct mp4_tag));
-	for (i = 0; i < metadata.count; i++) {
-		struct mp4_tag *tag = metadata.tags + i;
-
-		ret = -E_MP4_META_READ;
-		if (!mp4_meta_get_by_index(mp4, i, &tag->item, &tag->value))
-			goto close;
-		PARA_INFO_LOG("found: %s: %s\n", tag->item, tag->value);
-		if (!strcmp(tag->item, "artist"))
-			replace_tag(tag, tags->artist, &found_artist);
-		else if (!strcmp(tag->item, "title"))
-			replace_tag(tag, tags->title, &found_title);
-		else if (!strcmp(tag->item, "album"))
-			replace_tag(tag, tags->album, &found_album);
-		else if (!strcmp(tag->item, "date"))
-			replace_tag(tag, tags->year, &found_year);
-		else if (!strcmp(tag->item, "comment"))
-			replace_tag(tag, tags->comment, &found_comment);
-	}
-	if (!found_artist)
-		add_tag(&metadata, "artist", tags->artist);
-	if (!found_title)
-		add_tag(&metadata, "title", tags->title);
-	if (!found_album)
-		add_tag(&metadata, "album", tags->album);
-	if (!found_year)
-		add_tag(&metadata, "date", tags->year);
-	if (!found_comment)
-		add_tag(&metadata, "comment", tags->comment);
-	ret = -E_MP4_META_WRITE;
-	if (!mp4_meta_update(mp4, &metadata))
-		goto close;
-	ret = 1;
-close:
+	ret = mp4_open_meta(&cb, &mp4);
+	if (ret < 0)
+		return ret;
+	metadata = mp4_get_meta(mp4);
+	PARA_NOTICE_LOG("%u metadata item(s) found\n", metadata->count);
+	replace_or_add_tag("artist", tags->artist, metadata);
+	replace_or_add_tag("title", tags->title, metadata);
+	replace_or_add_tag("album", tags->album, metadata);
+	replace_or_add_tag("date", tags->year, metadata);
+	replace_or_add_tag("comment", tags->comment, metadata);
+	ret = mp4_update_meta(mp4);
 	mp4_close(mp4);
 	return ret;
 }