Remove unneccessary length checks in Code1

Revokes commit #024490 and re-establishes commit #0c00ec
Relating to #184 thanks to Milton Neal and Git Lost
This commit is contained in:
Robin Stuart 2020-04-01 19:27:49 +01:00
parent f02851b3c1
commit b4182cbc19

View File

@ -441,13 +441,11 @@ static int c1_encode(struct zint_symbol *symbol, unsigned char source[], unsigne
if (j == 13) { if (j == 13) {
latch = 0; latch = 0;
if ((length - sp) >= 14) {
for (i = sp + 13; i < length; i++) { for (i = sp + 13; i < length; i++) {
if (!((source[i] >= '0') && (source[i] <= '9'))) { if (!((source[i] >= '0') && (source[i] <= '9'))) {
latch = 1; latch = 1;
} }
} }
}
if (!(latch)) { if (!(latch)) {
next_mode = C1_DECIMAL; next_mode = C1_DECIMAL;
@ -492,13 +490,11 @@ static int c1_encode(struct zint_symbol *symbol, unsigned char source[], unsigne
if (j == 7) { if (j == 7) {
latch = 0; latch = 0;
if ((length - sp) >= 8) {
for (i = sp + 7; i < length; i++) { for (i = sp + 7; i < length; i++) {
if (!((source[sp + i] >= '0') && (source[sp + i] <= '9'))) { if (!((source[i] >= '0') && (source[i] <= '9'))) {
latch = 1; latch = 1;
} }
} }
}
if (!(latch)) { if (!(latch)) {
target[tp] = 236; /* FNC1 and change to Decimal */ target[tp] = 236; /* FNC1 and change to Decimal */