diff options
| author | Warren Dukes <warren.dukes@gmail.com> | 2004-10-16 12:22:19 +0000 |
|---|---|---|
| committer | Warren Dukes <warren.dukes@gmail.com> | 2004-10-16 12:22:19 +0000 |
| commit | 481a96ec40fccd5ab6025987adac0fbf83257bab (patch) | |
| tree | d61d036580a2bd5aea8fd3e35df9d5b1408b72c4 | |
| parent | c0e0c2460f19f750d7fb74f1f74d395c5a5c222c (diff) | |
| download | mpd-481a96ec40fccd5ab6025987adac0fbf83257bab.tar.gz mpd-481a96ec40fccd5ab6025987adac0fbf83257bab.tar.xz mpd-481a96ec40fccd5ab6025987adac0fbf83257bab.zip | |
#1) Fix a potential security hole, where mpd sends a line from a playlist to a client if the line is too long. But this is a rare circumstance, since the file would need to have a .m3u suffix and the line would need to be very long.
#2) Fix qball's bug, basically when we try todo initial buffering in decodeStart(), sleep for a few seconds after each attempt to finish init'ing the connection and begin buffering.
git-svn-id: https://svn.musicpd.org/mpd/trunk@2236 09075e82-0dd4-0310-85a5-a0d7c8717e4f
| -rw-r--r-- | src/decode.c | 6 | ||||
| -rw-r--r-- | src/playlist.c | 4 |
2 files changed, 8 insertions, 2 deletions
diff --git a/src/decode.c b/src/decode.c index 138162150..7296e154c 100644 --- a/src/decode.c +++ b/src/decode.c @@ -286,7 +286,11 @@ void decodeStart(PlayerControl * pc, OutputBuffer * cb, DecoderControl * dc) { dc->start = 0; while(!inputStreamAtEOF(&inStream) && bufferInputStream(&inStream) < 0 - && !dc->stop); + && !dc->stop) + { + /* sleep so we don't consume 100% of the cpu */ + my_usleep(1000); + } if(dc->stop) { dc->state = DECODE_STATE_STOP; diff --git a/src/playlist.c b/src/playlist.c index c7e5057db..fd1873f93 100644 --- a/src/playlist.c +++ b/src/playlist.c @@ -1398,7 +1398,9 @@ int loadPlaylist(FILE * fp, char * utf8file) { else if(slength==MAXPATHLEN) { s[slength] = '\0'; commandError(fp, ACK_ERROR_PLAYLIST_LOAD, - "\"%s\" too long", s); + "line in \"%s\" is too long", utf8file); + ERROR("line \"%s\" in playlist \"%s\" is too long\n", + s, utf8file); while(fclose(fileP) && errno==EINTR); if(erroredFile) free(erroredFile); return -1; |