diff options
| author | Ronald S. Bultje <rsbultje@gmail.com> | 2009-03-24 03:24:59 +0000 |
|---|---|---|
| committer | Ronald S. Bultje <rsbultje@gmail.com> | 2009-03-24 03:24:59 +0000 |
| commit | da1e126e0db3eda68153bb8130947adba8236034 (patch) | |
| tree | 4f2dcb5d8ce68d8abad652fe4d3577f94d14cf14 /libavformat | |
| parent | 29e455081fa8cd6d97b194c8dff33cc6473e33d9 (diff) | |
| download | ffmpeg-da1e126e0db3eda68153bb8130947adba8236034.tar.gz | |
strchr(string, '\0') returns non-NULL, and is thus not suited for use in
redir_isspace(char) to check if '\0' is a space or not. Therefore, we now
use memchr(), since then we can give the length of the string (i.e. the
length excluding the terminating '\0'). Fixes issue 919, see also the
follow-ups in the "[PATCH] rtsp.c small cleanups" mailinglist thread.
Originally committed as revision 18177 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavformat')
| -rw-r--r-- | libavformat/rtsp.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index c9e60dd59e..758314f097 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -55,7 +55,9 @@ static int rtsp_probe(AVProbeData *p) } #define SPACE_CHARS " \t\r\n" -#define redir_isspace(c) strchr(SPACE_CHARS, c) +/* we use memchr() instead of strchr() here because strchr() will return + * the terminating '\0' of SPACE_CHARS instead of NULL if c is '\0'. */ +#define redir_isspace(c) memchr(SPACE_CHARS, c, 4) static void skip_spaces(const char **pp) { const char *p; |