X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=aft.c;h=b4adbacf6e2a93b17c25f11bacb9845c47538c53;hp=e24a5fbfe1c490d1a79b691484f9d43fd60a6266;hb=e5e7cf0b78313ca5e696a56e8339c7090bf0cca0;hpb=bfb0f769d96dee610a25682b3b9c4218bc315b31 diff --git a/aft.c b/aft.c index e24a5fbf..b4adbacf 100644 --- a/aft.c +++ b/aft.c @@ -1035,24 +1035,26 @@ int open_and_update_audio_file(struct audio_file_data *afd) struct afsi_change_event_data aced; struct osl_object map, chunk_table_obj; struct ls_data *d = &status_item_ls_data; + unsigned char *tmp_hash; again: ret = score_get_best(¤t_aft_row, &d->score); if (ret < 0) return ret; /* * get_hash_of_row() and get_audio_file_path_of_row() initialize - * d->hash and d->path to point to memory-mapped files. These pointers + * their pointer argument to point to memory-mapped files. These pointers * become stale after a new audio file has been added or after the * server process received SIGHUP. For in both cases libosl unmaps and * remaps the underlying database files, and this remapping may well * change the starting address of the mapping. To avoid stale pointer * references we create copies on the heap. */ - free(d->hash); - ret = get_hash_of_row(current_aft_row, &d->hash); + ret = get_hash_of_row(current_aft_row, &tmp_hash); if (ret < 0) return ret; - d->hash = (unsigned char *)para_strdup((char *)d->hash); + if (!d->hash) + d->hash = para_malloc(HASH_SIZE); + memcpy(d->hash, tmp_hash, HASH_SIZE); free(d->path); ret = get_audio_file_path_of_row(current_aft_row, &d->path); if (ret < 0) @@ -2618,6 +2620,10 @@ static int aft_event_handler(enum afs_events event, struct para_buffer *pb, * current status items are affected and simply recreate them * every time. */ + ret = get_afhi_of_row(current_aft_row, + &status_item_ls_data.afhi); + if (ret < 0) + return ret; make_status_items(); return 0; } default: