]> granicus.if.org Git - openjpeg/commitdiff
[trunk] fixed a bug encoutntered when multiple MCT markers are provided
authorAntonin Descampe <antonin@gmail.com>
Tue, 20 Jan 2015 16:29:16 +0000 (16:29 +0000)
committerAntonin Descampe <antonin@gmail.com>
Tue, 20 Jan 2015 16:29:16 +0000 (16:29 +0000)
Update issue 430
Cc: savmickael@gmail.com
@Mickael: this revision affects your work on custom MCT. Is there a way
you could provide test data to add to our test suite ? Thanks

src/lib/openjp2/j2k.c

index 56077659d2a70c1279bff0a4901d6f3ab90be53e..f944ad1afb69a2e52abc41b52bedde0afb898961 100644 (file)
@@ -5236,6 +5236,7 @@ static OPJ_BOOL opj_j2k_read_mct (      opj_j2k_t *p_j2k,
                 }
 
                 l_mct_data = l_tcp->m_mct_records + l_tcp->m_nb_mct_records;
+                ++l_tcp->m_nb_mct_records;
         }
 
         if (l_mct_data->m_data) {
@@ -5264,7 +5265,6 @@ static OPJ_BOOL opj_j2k_read_mct (      opj_j2k_t *p_j2k,
         memcpy(l_mct_data->m_data,p_header_data,p_header_size);
 
         l_mct_data->m_data_size = p_header_size;
-        ++l_tcp->m_nb_mct_records;
 
         return OPJ_TRUE;
 }
@@ -7165,7 +7165,7 @@ OPJ_BOOL opj_j2k_read_header_procedure( opj_j2k_t *p_j2k,
 
                 /* Check if the current marker ID is valid */
                 if (l_current_marker < 0xff00) {
-                        opj_event_msg(p_manager, EVT_ERROR, "We expected read a marker ID (0xff--) instead of %.8x\n", l_current_marker);
+                        opj_event_msg(p_manager, EVT_ERROR, "A marker ID was expected (0xff--) instead of %.8x\n", l_current_marker);
                         return OPJ_FALSE;
                 }