[FFmpeg-devel] [PATCH] applehttp: Fix the key check in handle_variant_args

Ronald S. Bultje rsbultje
Mon Jan 24 19:08:44 CET 2011


Hi,

On Mon, Jan 24, 2011 at 4:11 AM, Martin Storsj? <martin at martin.st> wrote:
> The key string is supposed to contain the equals character,
> too. ?Since the checked string was wrong, and the return value
> check was wrong too, it incorrectly seemed to work right before.
> ---
> ?libavformat/applehttp.c | ? ?2 +-
> ?1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavformat/applehttp.c b/libavformat/applehttp.c
> index b73adcc..c35a1f1 100644
> --- a/libavformat/applehttp.c
> +++ b/libavformat/applehttp.c
> @@ -181,7 +181,7 @@ struct variant_info {
> ?static void handle_variant_args(struct variant_info *info, const char *key,
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? int key_len, char **dest, int *dest_len)
> ?{
> - ? ?if (strncmp(key, "BANDWIDTH", key_len)) {
> + ? ?if (!strncmp(key, "BANDWIDTH", key_len)) {
> ? ? ? ? *dest ? ? = ? ? ? ?info->bandwidth;
> ? ? ? ? *dest_len = sizeof(info->bandwidth);
> ? ? }
> --
> 1.7.3.1

You're not adding the '=' char to the string.

Ronald



More information about the ffmpeg-devel mailing list