diff mbox

[FFmpeg-devel] lavf/adp: Fix probe function on systems with signed char

Message ID CAB0OVGoxAkz3eZ1KDriwALfX_ZiCfLtEzW+gABfeScJ1PSkTRQ@mail.gmail.com
State Accepted
Headers show

Commit Message

Carl Eugen Hoyos Oct. 8, 2017, 9:42 p.m. UTC 
Hi!

Attached patch fixes the adp probe function, it currently fires for
repeated values.

I will probably apply soon, Carl Eugen

Comments

Carl Eugen Hoyos Oct. 9, 2017, 1:02 a.m. UTC | #1 
2017-10-08 23:42 GMT+02:00 Carl Eugen Hoyos <ceffmpeg@gmail.com>:
> Hi!
>
> Attached patch fixes the adp probe function, it currently fires for
> repeated values.

Pushed, Carl Eugen
diff mbox

Patch

From 33d39f5199bacbbbafdb9885c72c3c6898dded81 Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <ceffmpeg@gmail.com>
Date: Sun, 8 Oct 2017 23:41:14 +0200
Subject: [PATCH] lavf/adp: Fix the probe function on systems with signed
 char.

---
 libavformat/adp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/adp.c b/libavformat/adp.c
index 7355503..3914857 100644
--- a/libavformat/adp.c
+++ b/libavformat/adp.c
@@ -27,7 +27,7 @@ 
 static int adp_probe(AVProbeData *p)
 {
     int i, changes = 0;
-    char last = 0;
+    uint8_t last = 0;
 
     if (p->buf_size < 32)
         return 0;
-- 
1.7.10.4