[hw/scsi/scsi-bus.c:910]: (style) Expression '(X & 0x4) == 0x1' is always false. Source code is } else if ((buf[1] & 4) == 1) { Suggest code rework. I found this bug by running static analyser cppcheck over the source code. I also checked the latest code on the web and the bug exists there also.