[FFmpeg-devel] lavf/sdp: Change configuration pointer from char* to uint8_t*.

Hi!

Attached patch silences three warnings with clang and makes the
pointer type equal to what the function called with the pointer
expects.

Please comment, Carl Eugen

On Fri, Mar 15, 2019 at 12:56:05AM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch silences three warnings with clang and makes the
> pointer type equal to what the function called with the pointer
> expects.
> 
> Please comment, Carl Eugen

>  sdp.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 9f10029889140041ba33ed19e96c3a296f2edf85  0001-lavf-sdp-Change-pointer-to-configuration-from-char-t.patch
> From cf06b8cc09acd676ac6fb713168c23342d26e1b8 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
> Date: Fri, 15 Mar 2019 00:52:50 +0100
> Subject: [PATCH] lavf/sdp: Change pointer to configuration from char* to
>  uint8_t*.
> 
> This is also what av_base64_encode() expects.
> Fixes the following warnings with clang:
> libavformat/sdp.c:394:40: warning: implicit conversion from 'int' to 'char' changes value from 254 to -2
> libavformat/sdp.c:395:40: warning: implicit conversion from 'int' to 'char' changes value from 205 to -51
> libavformat/sdp.c:396:40: warning: implicit conversion from 'int' to 'char' changes value from 186 to -70
> ---
>  libavformat/sdp.c |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/sdp.c b/libavformat/sdp.c
> index a5d202e..34e9839 100644
> --- a/libavformat/sdp.c
> +++ b/libavformat/sdp.c
> @@ -347,7 +347,8 @@ static char *extradata2config(AVFormatContext *s, AVCodecParameters *par)
>  
>  static char *xiph_extradata2config(AVFormatContext *s, AVCodecParameters *par)
>  {
> -    char *config, *encoded_config;
> +    uint8_t *config;
> +    char *encoded_config;
>      const uint8_t *header_start[3];
>      int headers_len, header_len[3], config_len;
>      int first_header_size;

LGTM

thx

[...]

2019-03-16 0:11 GMT+01:00, Michael Niedermayer <michael@niedermayer.cc>:
> On Fri, Mar 15, 2019 at 12:56:05AM +0100, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch silences three warnings with clang and makes the
>> pointer type equal to what the function called with the pointer
>> expects.
>>
>> Please comment, Carl Eugen
>
>>  sdp.c |    3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 9f10029889140041ba33ed19e96c3a296f2edf85
>> 0001-lavf-sdp-Change-pointer-to-configuration-from-char-t.patch
>> From cf06b8cc09acd676ac6fb713168c23342d26e1b8 Mon Sep 17 00:00:00 2001
>> From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
>> Date: Fri, 15 Mar 2019 00:52:50 +0100
>> Subject: [PATCH] lavf/sdp: Change pointer to configuration from char* to
>>  uint8_t*.
>>
>> This is also what av_base64_encode() expects.
>> Fixes the following warnings with clang:
>> libavformat/sdp.c:394:40: warning: implicit conversion from 'int' to
>> 'char' changes value from 254 to -2
>> libavformat/sdp.c:395:40: warning: implicit conversion from 'int' to
>> 'char' changes value from 205 to -51
>> libavformat/sdp.c:396:40: warning: implicit conversion from 'int' to
>> 'char' changes value from 186 to -70
>> ---
>>  libavformat/sdp.c |    3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavformat/sdp.c b/libavformat/sdp.c
>> index a5d202e..34e9839 100644
>> --- a/libavformat/sdp.c
>> +++ b/libavformat/sdp.c
>> @@ -347,7 +347,8 @@ static char *extradata2config(AVFormatContext *s,
>> AVCodecParameters *par)
>>
>>  static char *xiph_extradata2config(AVFormatContext *s, AVCodecParameters
>> *par)
>>  {
>> -    char *config, *encoded_config;
>> +    uint8_t *config;
>> +    char *encoded_config;
>>      const uint8_t *header_start[3];
>>      int headers_len, header_len[3], config_len;
>>      int first_header_size;
>
> LGTM

Patch applied.

Thank you, Carl Eugen