This patchset fixes all the coverity issues for disas/: * dead ?: code in hppa * a NULL pointer deref following an internal-error case in i386 * unintended sign extension after shift of unsigned char in m68k, microblaze, cris, arm
The hppa, i386, and arm changes won't have a visible effect as they're either "never happens" cases or the code doesn't care. I think m68k, microblaze and cris are genuinely bug fixes. Disclaimer: generally compile tested only. thanks -- PMM Peter Maydell (6): disas/hppa: Remove dead code disas/i386: Avoid NULL pointer dereference in error case disas/m68k: Avoid unintended sign extension in get_field() disas/microblaze: Avoid unintended sign extension disas/cris: Avoid unintended sign extension disas/arm: Avoid unintended sign extension disas/arm.c | 4 ++-- disas/cris.c | 4 ++-- disas/hppa.c | 3 +-- disas/i386.c | 2 +- disas/m68k.c | 5 +++-- disas/microblaze.c | 6 ++++-- 6 files changed, 13 insertions(+), 11 deletions(-) -- 2.7.4
