[vlc-devel] [PATCH 1/2] demux: adaptive: fix out of range exception

[Jean-Baptiste Kempf]

Does this patch close a known bug? Can you mention it in the commit log?

On Thu, 12 Oct 2017, at 01:11, Zhao Zhili wrote:
> 
> ---
>  modules/demux/adaptive/http/HTTPConnection.cpp | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/modules/demux/adaptive/http/HTTPConnection.cpp
> b/modules/demux/adaptive/http/HTTPConnection.cpp
> index 5934fb7270..a1f81b61d8 100644
> --- a/modules/demux/adaptive/http/HTTPConnection.cpp
> +++ b/modules/demux/adaptive/http/HTTPConnection.cpp
> @@ -246,10 +246,9 @@ int HTTPConnection::parseReply()
>          size_t split = lines.find_first_of(':');
>          if(split != std::string::npos)
>          {
> -            size_t value = split + 1;
> -            while(lines.at(value) == ' ')
> -                value++;
> -
> +            size_t value = lines.find_first_not_of(' ', split + 1);
> +            if(value == std::string::npos)
> +                value = lines.length();
>              onHeader(lines.substr(0, split), lines.substr(value));
>              lines = std::string();
>          }
> -- 
> 2.14.2
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
> Email had 1 attachment:
> + 0001-demux-adaptive-fix-out-of-range-exception.patch
>   2k (text/plain)


-- 
Jean-Baptiste Kempf -  President
+33 672 704 734