check pointer overflow in t2.c

core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c

Index: core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c
diff --git a/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c b/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c
index dd188c1ba73b4bd08f63572693d54f1f9c9f3af5..683d0415cf0fe8231bfe7858f55daa80d98df37c 100644
--- a/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c
+++ b/core/src/fxcodec/fx_libopenjpeg/libopenjpeg20/jp2.c
@@ -1280,7 +1280,7 @@ OPJ_BOOL opj_jp2_read_colr( opj_jp2_t *jp2,
 	}
 	else if (jp2->meth > 2)
     {
-        /*	ISO/IEC 15444-1:2004 (E), Table I.9 ­ Legal METH values:
+        /*	ISO/IEC 15444-1:2004 (E), Table I.9  Legal METH values:

[Tom Sepez, 2014/10/03 18:41:10]

nit: looks to be an unintended whitespace only change.

[Bo Xu, 2014/10/03 19:08:38]

I got the warning of "the file contains a character that cannot be represented
in the current code page (936)" when I compiled in VS2013. Seems there is the
char that brought in from the openjpeg update.

         conforming JP2 reader shall ignore the entire Colour Specification box.*/
         opj_event_msg(p_manager, EVT_INFO, "COLR BOX meth value is not a regular value (%d), " 
             "so we will ignore the entire Colour Specification box. \n", jp2->meth);