# HG changeset patch # User Matti Hamalainen # Date 1530138310 -10800 # Node ID 1c5ea4fa8788607f7512c7f9048d9445349100d0 # Parent 42cd527a01b9fc2d05496c8724d55e441455e40c Add some default-case error handling. diff -r 42cd527a01b9 -r 1c5ea4fa8788 tools/lib64gfx.c --- a/tools/lib64gfx.c Thu Jun 28 01:24:38 2018 +0300 +++ b/tools/lib64gfx.c Thu Jun 28 01:25:10 2018 +0300 @@ -971,6 +971,12 @@ case DO_SET_MEM_LO: value = *src & 0x0f; break; + + default: + return dmError(DMERR_INTERNAL, + "Unhandled op type %d in " + "op #%d, offs=%d ($%04x), bank=%d, size=%d ($%04x) @ %d ($%04x)\n", + op->type, i, op->offs, op->offs, op->bank, size, size, buf->len, buf->len); } switch (op->subject) { @@ -1133,6 +1139,11 @@ case DO_SET_OP: // This operation makes no sense in encoding, so do nothing break; + default: + return dmError(DMERR_INTERNAL, + "Unhandled op type %d in " + "op #%d, offs=%d ($%04x), bank=%d, size=%d ($%04x) @ %d ($%04x)\n", + op->type, i, op->offs, op->offs, op->bank, size, size, buf->len, buf->len); } break; @@ -1169,6 +1180,12 @@ case DO_SET_OP: // Do nothing in this case break; + + default: + return dmError(DMERR_INTERNAL, + "Unhandled op type %d in " + "op #%d, offs=%d ($%04x), bank=%d, size=%d ($%04x) @ %d ($%04x)\n", + op->type, i, op->offs, op->offs, op->bank, size, size, buf->len, buf->len); } break;