diff options
| author | Corey Minyard | 2016-01-11 14:32:32 +0100 |
|---|---|---|
| committer | Paolo Bonzini | 2016-02-09 15:46:54 +0100 |
| commit | 37eebb8693368d890b700cca6e39ec31c7e980e5 (patch) | |
| tree | 62cfca8da7c77083caae7c2e4affacad283ecd69 /include/qapi/visitor-impl.h | |
| parent | ipmi_bmc_sim: Fix off by one in check. (diff) | |
| download | qemu-37eebb8693368d890b700cca6e39ec31c7e980e5.tar.gz qemu-37eebb8693368d890b700cca6e39ec31c7e980e5.tar.xz qemu-37eebb8693368d890b700cca6e39ec31c7e980e5.zip | |
ipmi_bmc_sim: Add break to correct watchdog NMI check
It was falling through when it should have been a break. Found by
Coverity. The logic could be simplified a bit with a fallthrough,
probably the original thought, but that would be less clear, I think.
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: Shannon Zhao <zhaoshenglong@huawei.com>
Cc: Xiao Guangrong <guangrong.xiao@linux.intel.com>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Message-Id: <1452519152-6500-3-git-send-email-minyard@acm.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'include/qapi/visitor-impl.h')
0 files changed, 0 insertions, 0 deletions