Config path functions can return NULL

It's quite unlikely, but functions like mp_find_user_config_file() can
return NULL, e.g. if $HOME is unset.

Fix all the code that didn't check for this correctly yet.

1 files changed, 5 insertions, 3 deletions

diff --git a/mpvcore/mplayer.c b/mpvcore/mplayer.c
index 7c8d33a6cd..1f44ac9b7d 100644
--- a/mpvcore/mplayer.c
+++ b/mpvcore/mplayer.c
@@ -628,9 +628,11 @@ static void mk_config_dir(char *subdir)
 {
     void *tmp = talloc_new(NULL);
     char *confdir = talloc_steal(tmp, mp_find_user_config_file(""));
-    if (subdir)
-        confdir = mp_path_join(tmp, bstr0(confdir), bstr0(subdir));
-    mkdir(confdir, 0777);
+    if (confdir) {
+        if (subdir)
+            confdir = mp_path_join(tmp, bstr0(confdir), bstr0(subdir));
+        mkdir(confdir, 0777);
+    }
     talloc_free(tmp);
 }