diff options
| author | Max Kellermann <max@duempel.org> | 2008-10-08 11:06:37 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2008-10-08 11:06:37 +0200 |
| commit | e1a4474ad4a0bc78ac338223b0528e0161f38e98 (patch) | |
| tree | 86dddab67bc2f103827862bc887e643c54332557 /src | |
| parent | 4ab6b59aaf9db6d6108b0734da2307f7fac7fb4b (diff) | |
| download | mpd-e1a4474ad4a0bc78ac338223b0528e0161f38e98.tar.gz mpd-e1a4474ad4a0bc78ac338223b0528e0161f38e98.tar.xz mpd-e1a4474ad4a0bc78ac338223b0528e0161f38e98.zip | |
song: don't allow calling song_get_url(NULL)
The runtime check suggests that the author has somehow thought
song_get_url(NULL) might be valid. It should not be. Replace it with
an assertion.
Diffstat (limited to 'src')
| -rw-r--r-- | src/decoder_thread.c | 5 | ||||
| -rw-r--r-- | src/song.c | 4 |
2 files changed, 2 insertions, 7 deletions
diff --git a/src/decoder_thread.c b/src/decoder_thread.c index a8ff2541b..594bfbc77 100644 --- a/src/decoder_thread.c +++ b/src/decoder_thread.c @@ -37,10 +37,7 @@ static void decodeStart(void) char path_max_fs[MPD_PATH_MAX]; char path_max_utf8[MPD_PATH_MAX]; - if (!song_get_url(dc.next_song, path_max_utf8)) { - dc.error = DECODE_ERROR_FILE; - goto stop_no_close; - } + song_get_url(dc.next_song, path_max_utf8); if (!isRemoteUrl(path_max_utf8)) { rmp2amp_r(path_max_fs, utf8_to_fs_charset(path_max_fs, path_max_utf8)); diff --git a/src/song.c b/src/song.c index bf4be4901..5653e045d 100644 --- a/src/song.c +++ b/src/song.c @@ -121,9 +121,7 @@ song_file_update(struct song *song) char * song_get_url(struct song *song, char *path_max_tmp) { - if (!song) - return NULL; - + assert(song != NULL); assert(*song->url); if (!song->parent || !song->parent->path) |