X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=aft.c;h=d5a355566e387df5bc6e8b1ec06fad03bb660c3e;hp=1b23823e77dd34bcc7acd11dba6e6eca2d56d7db;hb=dded41e5bed40cca8876e1056351cc1dbab309d0;hpb=c8d6b9290983715325da4f6453d13c6d53e1d418

diff --git a/aft.c b/aft.c
index 1b23823e..d5a35556 100644
--- a/aft.c
+++ b/aft.c
@@ -20,6 +20,7 @@
 #include "vss.h"
 #include "fd.h"
 #include "ipc.h"
+#include "portable_io.h"
 
 static struct osl_table *audio_file_table;
 
@@ -588,8 +589,6 @@ int get_afhi_of_row(const struct osl_row *row, struct afh_info *afhi)
 static int save_afd(struct audio_file_data *afd)
 {
 	size_t size = sizeof(*afd) + sizeof_chunk_table(&afd->afhi);
-
-	PARA_DEBUG_LOG("size: %zu\n", size);
 	int shmid, ret = shm_new(size);
 	void *shm_afd;
 	char *buf;
@@ -1261,8 +1260,8 @@ static int prepare_ls_row(struct osl_row *row, void *ls_opts)
 	GET_NUM_DIGITS(d->afsi.num_played, &num_digits);
 	w->num_played_width = PARA_MAX(w->num_played_width, num_digits);
 	/* get the number of chars to print this amount of time */
-	tmp = get_duration_width(d->afhi.seconds_total);
-	w->duration_width = PARA_MAX(w->duration_width, tmp);
+	num_digits = get_duration_width(d->afhi.seconds_total);
+	w->duration_width = PARA_MAX(w->duration_width, num_digits);
 	GET_NUM_DIGITS(d->afsi.amp, &num_digits);
 	w->amp_width = PARA_MAX(w->amp_width, num_digits);
 	if (options->flags & LS_FLAG_ADMISSIBLE_ONLY) {
@@ -1599,6 +1598,7 @@ static void com_add_callback(int fd, const struct osl_object *query)
 	struct afs_info default_afsi = {.last_played = 0};
 	struct para_buffer msg = {.max_size = SHMMAX,
 		.max_size_handler = pass_buffer_as_shm, .private_data = &fd};
+	uint16_t afhi_offset, chunks_offset;
 
 	hash = (HASH_TYPE *)buf + CAB_HASH_OFFSET;
 	hash_to_asc(hash, asc);;
@@ -1653,8 +1653,8 @@ static void com_add_callback(int fd, const struct osl_object *query)
 			goto out;
 	}
 	/* no hs or force mode, child must have sent afhi */
-	uint16_t afhi_offset = read_u16(buf + CAB_AFHI_OFFSET_POS);
-	uint16_t chunks_offset = read_u16(buf + CAB_CHUNKS_OFFSET_POS);
+	afhi_offset = read_u16(buf + CAB_AFHI_OFFSET_POS);
+	chunks_offset = read_u16(buf + CAB_CHUNKS_OFFSET_POS);
 
 	objs[AFTCOL_AFHI].data = buf + afhi_offset;
 	objs[AFTCOL_AFHI].size = chunks_offset - afhi_offset;
@@ -1816,6 +1816,7 @@ static int add_one_audio_file(const char *path, void *private_data)
 		afhi_ptr = &afhi;
 	}
 	munmap(map.data, map.size);
+	close(fd);
 	if (pad->flags & ADD_FLAG_VERBOSE) {
 		send_ret = send_va_buffer(pad->fd, "adding %s\n", path);
 		if (send_ret < 0)
@@ -2303,10 +2304,6 @@ static void com_cpsi_callback(int fd, const struct osl_object *query)
 	};
 	int ret;
 	char *source_path = (char *)query->data + sizeof(cad.flags);
-
-	ret = get_afsi_of_path(source_path, &cad.source_afsi);
-	if (ret < 0)
-		goto out;
 	struct pattern_match_data pmd = {
 		.table = audio_file_table,
 		.loop_col_num = AFTCOL_HASH,
@@ -2317,6 +2314,10 @@ static void com_cpsi_callback(int fd, const struct osl_object *query)
 		.data = &cad,
 		.action = copy_selector_info
 	};
+
+	ret = get_afsi_of_path(source_path, &cad.source_afsi);
+	if (ret < 0)
+		goto out;
 	ret = for_each_matching_row(&pmd);
 out:
 	if (ret < 0)