From 38874b2f2e3a5e0473bada789ef04af632e27ecb Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 12 Oct 2013 17:19:20 +0200 Subject: playlist: check for NULL on caller site Letting some playlist functions randomly accept NULL, while others do not isn't such a good idea. --- mpvcore/playlist.c | 2 +- mpvcore/playlist_parser.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/mpvcore/playlist.c b/mpvcore/playlist.c index 818aa45a8c..3396a5d78b 100644 --- a/mpvcore/playlist.c +++ b/mpvcore/playlist.c @@ -178,7 +178,7 @@ struct playlist_entry *playlist_get_next(struct playlist *pl, int direction) void playlist_add_base_path(struct playlist *pl, bstr base_path) { - if (!pl || base_path.len == 0 || bstrcmp0(base_path, ".") == 0) + if (base_path.len == 0 || bstrcmp0(base_path, ".") == 0) return; for (struct playlist_entry *e = pl->first; e; e = e->next) { if (!mp_is_url(bstr0(e->filename))) { diff --git a/mpvcore/playlist_parser.c b/mpvcore/playlist_parser.c index 0b3be14716..7c77760df4 100644 --- a/mpvcore/playlist_parser.c +++ b/mpvcore/playlist_parser.c @@ -648,7 +648,8 @@ struct playlist *playlist_parse_file(const char *file, struct MPOpts *opts) struct playlist *ret = do_parse(stream, true); free_stream(stream); - playlist_add_base_path(ret, mp_dirname(file)); + if (ret) + playlist_add_base_path(ret, mp_dirname(file)); return ret; -- cgit v1.2.3