Add a sanity check to unsafe-looking code.

Does the memcpy to internal buffer here have any purpose besides
enabling more potential buffer overflows?


git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@21250 b3059339-0415-0410-9bf9-f77b7e298cf2

1 files changed, 2 insertions, 0 deletions

diff --git a/sub_cc.c b/sub_cc.c
index 2e3123f7bf..28c4bdb2c3 100644
--- a/sub_cc.c
+++ b/sub_cc.c
@@ -284,6 +284,8 @@ static void subcc_decode(void)
 
 void subcc_process_data(unsigned char *inputdata,unsigned int len)
 {
+	if (len > CC_INPUTBUFFER_SIZE)
+		return;
 	if(!subcc_enabled) return;
 	if(!inited) subcc_init();