diff options
| author | Paolo Bonzini | 2016-07-15 17:29:22 +0200 |
|---|---|---|
| committer | Michael Tokarev | 2016-09-13 17:12:34 +0200 |
| commit | 5ce747cfac697f61668ab4fa4a71c1dba15cc272 (patch) | |
| tree | fe2aadeb7207aa7300a43bfb80f95356e83c7e14 | |
| parent | timer/cpus: fix some typos and update some comments (diff) | |
| download | qemu-5ce747cfac697f61668ab4fa4a71c1dba15cc272.tar.gz qemu-5ce747cfac697f61668ab4fa4a71c1dba15cc272.tar.xz qemu-5ce747cfac697f61668ab4fa4a71c1dba15cc272.zip | |
target-m68k: fix get_mac_extf helper
val is assigned twice; the second one should be combined with "|".
Reported by Coverity.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
| -rw-r--r-- | target-m68k/helper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/target-m68k/helper.c b/target-m68k/helper.c index f52d0e3036..89bbe6dfa6 100644 --- a/target-m68k/helper.c +++ b/target-m68k/helper.c @@ -812,7 +812,7 @@ uint32_t HELPER(get_mac_extf)(CPUM68KState *env, uint32_t acc) { uint32_t val; val = env->macc[acc] & 0x00ff; - val = (env->macc[acc] >> 32) & 0xff00; + val |= (env->macc[acc] >> 32) & 0xff00; val |= (env->macc[acc + 1] << 16) & 0x00ff0000; val |= (env->macc[acc + 1] >> 16) & 0xff000000; return val; |