update: validate in command.c and fix small memory leak

If update_task was called with "" as its path argument, that
string would never get freed because it false positived.

Instead, never pass "" to update_task and trip an assertion
if we do.

1 files changed, 8 insertions, 3 deletions

diff --git a/src/update.c b/src/update.c
index 72f4b943b..6c8b6e7bc 100644
--- a/src/update.c
+++ b/src/update.c
@@ -349,9 +349,12 @@ static void updatePath(const char *utf8path)
 
 static void * update_task(void *_path)
 {
-	if (_path && !path_is_music_root(_path)) {
-		updatePath(_path);
-		free(_path);
+	char *utf8path = _path;
+
+	if (utf8path) {
+		assert(*utf8path);
+		updatePath(utf8path);
+		free(utf8path);
 	} else {
 		struct stat st;
 
@@ -386,6 +389,8 @@ unsigned directory_update_init(char *path)
 {
 	assert(pthread_equal(pthread_self(), main_task));
 
+	assert(!path || (path && *path));
+
 	if (progress != UPDATE_PROGRESS_IDLE) {
 		unsigned next_task_id;