MyFavMirrors/owntone-server
1
0
Fork 0
mirror of https://github.com/owntone/owntone-server.git synced 2024-12-26 23:25:56 -05:00
Code Issues Packages Projects Releases Wiki Activity

[jsonapi] Fix segfault if no default playlist directory is set in conf

Browse Source
This commit is contained in:
chme 2019-06-01 08:02:46 +02:00
parent 4f5966c9ff
commit d35333139e
1 changed files with 29 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
src/httpd_jsonapi.c
View File

@ -59,8 +59,9 @@
# include "spotify.h" # include "spotify.h"
#endif #endif
static bool allow_modifying_stored_playlists; static bool allow_modifying_stored_playlists;
static char *default_pl_dir; static char *default_playlist_directory;
/* -------------------------------- HELPERS --------------------------------- */ /* -------------------------------- HELPERS --------------------------------- */
@ -713,8 +714,8 @@ jsonapi_reply_config(struct httpd_request *hreq)
json_object_object_add(jreply, "directories", directories); json_object_object_add(jreply, "directories", directories);
// Config for creating/modifying stored playlists // Config for creating/modifying stored playlists
json_object_object_add(jreply, "allow_modifying_stored_playlists", json_object_new_boolean(cfg_getbool(lib, "allow_modifying_stored_playlists"))); json_object_object_add(jreply, "allow_modifying_stored_playlists", json_object_new_boolean(allow_modifying_stored_playlists));
json_object_object_add(jreply, "default_playlist_directory", json_object_new_string(cfg_getstr(lib, "default_playlist_directory"))); safe_json_add_string(jreply, "default_playlist_directory", default_playlist_directory);
CHECK_ERRNO(L_WEB, evbuffer_add_printf(hreq->reply, "%s", json_object_to_json_string(jreply))); CHECK_ERRNO(L_WEB, evbuffer_add_printf(hreq->reply, "%s", json_object_to_json_string(jreply)));
@ -2929,13 +2930,13 @@ jsonapi_reply_queue_save(struct httpd_request *hreq)
if (!allow_modifying_stored_playlists) if (!allow_modifying_stored_playlists)
{ {
DPRINTF(E_LOG, L_WEB, "Saving playlists disabled in cfg, ignoring request\n"); DPRINTF(E_LOG, L_WEB, "Modifying stored playlists is not enabled in the config file\n");
return 403; return 403;
} }
if (access(default_pl_dir, W_OK) < 0) if (access(default_playlist_directory, W_OK) < 0)
{ {
DPRINTF(E_LOG, L_WEB, "Invalid playlist save directory=%s\n", default_pl_dir); DPRINTF(E_LOG, L_WEB, "Invalid playlist save directory '%s'\n", default_playlist_directory);
return 403; return 403;
} }
@ -2948,7 +2949,7 @@ jsonapi_reply_queue_save(struct httpd_request *hreq)
return HTTP_BADREQUEST; return HTTP_BADREQUEST;
} }
snprintf(buf, sizeof(buf), "/file:%s/%s", default_pl_dir, playlist_name); snprintf(buf, sizeof(buf), "/file:%s/%s", default_playlist_directory, playlist_name);
free(playlist_name); free(playlist_name);
ret = library_queue_save(buf); ret = library_queue_save(buf);
@ -3645,6 +3646,7 @@ int
jsonapi_init(void) jsonapi_init(void)
{ {
char buf[64]; char buf[64];
char *temp_path;
int i; int i;
int ret; int ret;
@ -3660,19 +3662,26 @@ jsonapi_init(void)
} }
} }
default_pl_dir = NULL; default_playlist_directory = NULL;
allow_modifying_stored_playlists = cfg_getbool(cfg_getsec(cfg, "library"), "allow_modifying_stored_playlists"); allow_modifying_stored_playlists = cfg_getbool(cfg_getsec(cfg, "library"), "allow_modifying_stored_playlists");
if (allow_modifying_stored_playlists) if (allow_modifying_stored_playlists)
{ {
default_pl_dir = cfg_getstr(cfg_getsec(cfg, "library"), "default_playlist_directory"); temp_path = cfg_getstr(cfg_getsec(cfg, "library"), "default_playlist_directory");
if (default_pl_dir == NULL) if (temp_path)
{ {
allow_modifying_stored_playlists = false; // The path in the conf file may have a trailing slash character. Return the realpath like it is done for the library directories.
DPRINTF(E_LOG, L_WEB, "Invalid playlist save directory, disabling\n"); default_playlist_directory = realpath(temp_path, NULL);
if (default_playlist_directory)
{
if (access(default_playlist_directory, W_OK) < 0)
DPRINTF(E_WARN, L_WEB, "Non-writable playlist save directory '%s'\n", default_playlist_directory);
} }
else if (access(default_pl_dir, W_OK) < 0) }
if (!default_playlist_directory)
{ {
DPRINTF(E_WARN, L_WEB, "Non-writable playlist save directory=%s\n", default_pl_dir); DPRINTF(E_LOG, L_WEB, "Invalid playlist save directory, disabling modifying stored playlists\n");
allow_modifying_stored_playlists = false;
} }
} }
@ -3686,4 +3695,6 @@ jsonapi_deinit(void)
for (i = 0; adm_handlers[i].handler; i++) for (i = 0; adm_handlers[i].handler; i++)
regfree(&adm_handlers[i].preg); regfree(&adm_handlers[i].preg);
free(default_playlist_directory);
} }