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

Carl Eugen Hoyos git at videolan.org
Wed Mar 20 18:19:33 EET 2019


ffmpeg | branch: master | Carl Eugen Hoyos <ceffmpeg at gmail.com> | Fri Mar 15 00:52:50 2019 +0100| [a171cafb355935faf563f979a2382d98dfb0b8c4] | committer: Carl Eugen Hoyos

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

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a171cafb355935faf563f979a2382d98dfb0b8c4
---

 libavformat/sdp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libavformat/sdp.c b/libavformat/sdp.c
index a5d202e99c..34e9839b67 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;



More information about the ffmpeg-cvslog mailing list