# HG changeset patch # User ramiro # Date 1238384560 0 # Node ID e02f401dde04e8ea7b3e8cf5d810e4844221fd15 # Parent f2f4f136add4cfa58a061cf85fe9f47c914a1361 mlpdec: Simplify substream length mismatch error. diff -r f2f4f136add4 -r e02f401dde04 mlpdec.c --- a/mlpdec.c Mon Mar 30 03:37:19 2009 +0000 +++ b/mlpdec.c Mon Mar 30 03:42:40 2009 +0000 @@ -1017,8 +1017,10 @@ if (read_block_data(m, &gb, substr) < 0) return -1; - } while ((get_bits_count(&gb) < substream_data_len[substr] * 8) - && get_bits1(&gb) == 0); + if (get_bits_count(&gb) >= substream_data_len[substr] * 8) + goto substream_length_mismatch; + + } while (!get_bits1(&gb)); skip_bits(&gb, (-get_bits_count(&gb)) & 15); if (substream_data_len[substr] * 8 - get_bits_count(&gb) >= 32) { @@ -1049,9 +1051,7 @@ av_log(m->avctx, AV_LOG_ERROR, "Substream %d checksum failed.\n" , substr); } if (substream_data_len[substr] * 8 != get_bits_count(&gb)) { - av_log(m->avctx, AV_LOG_ERROR, "substream %d length mismatch\n", - substr); - return -1; + goto substream_length_mismatch; } next_substr: @@ -1065,6 +1065,10 @@ return length; +substream_length_mismatch: + av_log(m->avctx, AV_LOG_ERROR, "substream %d length mismatch\n", substr); + return -1; + error: m->params_valid = 0; return -1;