diff options
| author | Max Kellermann <max@duempel.org> | 2012-06-13 21:23:34 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2012-06-13 21:26:32 +0200 |
| commit | 9f3db5a70b4181fef6a70ef84b70fdd5b3c3e451 (patch) | |
| tree | 2e98f6d2adb18cb97320a4622bd5869d42a34c69 | |
| parent | 1a59730782bffe91d4d136cee5c65f9f5726bb75 (diff) | |
directory: require db lock for _{add,remove}_song()
| -rw-r--r-- | src/directory.c | 2 | ||||
| -rw-r--r-- | src/update_walk.c | 8 |
2 files changed, 10 insertions, 0 deletions
diff --git a/src/directory.c b/src/directory.c index 93088112..e886698d 100644 --- a/src/directory.c +++ b/src/directory.c @@ -186,6 +186,7 @@ directory_lookup_directory(struct directory *directory, const char *uri) void directory_add_song(struct directory *directory, struct song *song) { + assert(holding_db_lock()); assert(directory != NULL); assert(song != NULL); assert(song->parent == directory); @@ -197,6 +198,7 @@ void directory_remove_song(G_GNUC_UNUSED struct directory *directory, struct song *song) { + assert(holding_db_lock()); assert(directory != NULL); assert(song != NULL); assert(song->parent == directory); diff --git a/src/update_walk.c b/src/update_walk.c index 0fcfaf37..6a953f52 100644 --- a/src/update_walk.c +++ b/src/update_walk.c @@ -236,7 +236,10 @@ update_archive_tree(struct directory *directory, char *name) if (song == NULL) { song = song_file_load(name, directory); if (song != NULL) { + db_lock(); directory_add_song(directory, song); + db_unlock(); + modified = true; g_message("added %s/%s", directory_get_path(directory), name); @@ -374,7 +377,9 @@ update_container_file(struct directory *directory, &add_tag_handler, song->tag); g_free(child_path_fs); + db_lock(); directory_add_song(contdir, song); + db_unlock(); modified = true; @@ -438,7 +443,10 @@ update_song_file(struct directory *directory, return; } + db_lock(); directory_add_song(directory, song); + db_unlock(); + modified = true; g_message("added %s/%s", directory_get_path(directory), name); |