[MPlayer-cvslog] r21250 - trunk/sub_cc.c

uau subversion at mplayerhq.hu
Sat Nov 25 21:01:25 CET 2006


Author: uau
Date: Sat Nov 25 21:01:24 2006
New Revision: 21250

Modified:
   trunk/sub_cc.c

Log:
Add a sanity check to unsafe-looking code.
Does the memcpy to internal buffer here have any purpose besides
enabling more potential buffer overflows?


Modified: trunk/sub_cc.c
==============================================================================
--- trunk/sub_cc.c	(original)
+++ trunk/sub_cc.c	Sat Nov 25 21:01:24 2006
@@ -284,6 +284,8 @@
 
 void subcc_process_data(unsigned char *inputdata,unsigned int len)
 {
+	if (len > CC_INPUTBUFFER_SIZE)
+		return;
 	if(!subcc_enabled) return;
 	if(!inited) subcc_init();
 	



More information about the MPlayer-cvslog mailing list