http: fix potentially dangerous whitespace skipping code
authorwm4 <nfxjfg@googlemail.com>
Thu, 8 Mar 2018 03:52:36 +0000 (04:52 +0100)
committerwm4 <nfxjfg@googlemail.com>
Sun, 18 Mar 2018 11:36:24 +0000 (12:36 +0100)
commitb7d842c554b1fec051ca906f446f7311139c5725
tree4b47cc1573159597bdd91cd9e2450bcab98de754
parentc0687acbf6094053834af6a20e9d71b455842c8c
http: fix potentially dangerous whitespace skipping code

If the string consists entirely of whitespace, this could in theory
continue to write '\0' before the start of the memory allocation. In
practice, it didn't really happen: the generic HTTP header parsing code
already skips leading whitespaces, so the string is either empty, or
consists a non-whitespace. (The generic code and the cookie code
actually have different ideas about what bytes are whitespace: the
former uses av_isspace(), the latter uses WHITESPACES. Fortunately,
av_isspace() is a super set of the http.c specific WHITESPACES, so
there's probably no case where the above assumption could have been
broken.)
libavformat/http.c