X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=aft.c;h=b358c96cd9afd23bfa7f043c8add8bbf8b4bfade;hp=b4adbacf6e2a93b17c25f11bacb9845c47538c53;hb=c0e4799bafc72ffdb7e8912317b8f74acbbc2736;hpb=b3c68fd8519a426d6b6285dcb5a865670285e99a

diff --git a/aft.c b/aft.c
index b4adbacf..b358c96c 100644
--- a/aft.c
+++ b/aft.c
@@ -407,7 +407,7 @@ static void load_chunk_table(struct afh_info *afhi, const struct osl_object *ct)
 	int i;
 	size_t sz;
 
-	if (!ct->data || ct->size < 4) {
+	if (!ct->data || ct->size < 4 * (afhi->chunks_total + 1)) {
 		afhi->chunk_table = NULL;
 		return;
 	}
@@ -589,8 +589,10 @@ static int get_hash_of_row(const struct osl_row *row, unsigned char **hash)
 int get_afhi_of_row(const struct osl_row *row, struct afh_info *afhi)
 {
 	struct osl_object obj;
-	int ret = osl(osl_get_object(audio_file_table, row, AFTCOL_AFHI,
-		&obj));
+	int ret;
+
+	assert(row);
+	ret = osl(osl_get_object(audio_file_table, row, AFTCOL_AFHI, &obj));
 	if (ret < 0)
 		return ret;
 	load_afhi(obj.data, afhi);
@@ -719,7 +721,8 @@ __a_const static short unsigned get_duration_width(int seconds)
 	return width + 6;
 }
 
-static void get_duration_buf(int seconds, char *buf, struct ls_options *opts)
+static void get_duration_buf(int seconds, char *buf, size_t bufsize,
+		struct ls_options *opts)
 {
 	unsigned hours = seconds / 3600, mins = (seconds % 3600) / 60;
 	short unsigned max_width;
@@ -727,10 +730,12 @@ static void get_duration_buf(int seconds, char *buf, struct ls_options *opts)
 	if (!hours) { /* m:ss or mm:ss */
 		max_width = opts->mode == LS_MODE_LONG?
 			opts->widths.duration_width : 4;
+		assert(max_width < bufsize - 1);
 		sprintf(buf, "%*u:%02d", max_width - 3, mins, seconds % 60);
 	} else { /* more than one hour => h:mm:ss, hh:mm:ss, hhh:mm:ss, ... */
 		max_width = opts->mode == LS_MODE_LONG?
 			opts->widths.duration_width : 7;
+		assert(max_width < bufsize - 1);
 		sprintf(buf, "%*u:%02u:%02d", max_width - 6, hours, mins,
 			seconds % 60);
 	}
@@ -854,7 +859,8 @@ static int print_list_item(struct ls_data *d, struct ls_options *opts,
 		if (ret < 0)
 			goto out;
 	}
-	get_duration_buf(afhi->seconds_total, duration_buf, opts);
+	get_duration_buf(afhi->seconds_total, duration_buf,
+		sizeof(duration_buf), opts);
 	if (opts->mode == LS_MODE_LONG) {
 		struct ls_widths *w = &opts->widths;
 		if (lls_opt_given(r_a))
@@ -2534,10 +2540,12 @@ static int aft_open(const char *dir)
 		PARA_NOTICE_LOG("current audio file hash lookup: success\n");
 		return 1;
 	}
-	PARA_NOTICE_LOG("failed to open audio file table\n");
 	audio_file_table = NULL;
-	if (ret == -OSL_ERRNO_TO_PARA_ERROR(E_OSL_NOENT))
+	if (ret == -OSL_ERRNO_TO_PARA_ERROR(E_OSL_NOENT)) {
+		PARA_WARNING_LOG("no audio file table\n");
 		return 1;
+	}
+	PARA_NOTICE_LOG("failed to open audio file table\n");
 	return ret;
 }
 
@@ -2618,8 +2626,10 @@ static int aft_event_handler(enum afs_events event, struct para_buffer *pb,
 		/*
 		 * These events are rare. We don't bother to check whether the
 		 * current status items are affected and simply recreate them
-		 * every time.
+		 * whenever an audio file is open.
 		 */
+		if (!current_aft_row)
+			return 0;
 		ret = get_afhi_of_row(current_aft_row,
 			&status_item_ls_data.afhi);
 		if (ret < 0)