summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorbalrog <balrog@c046a42c-6fe2-441c-8c8c-71466251a162>2007-11-18 02:09:36 +0000
committerbalrog <balrog@c046a42c-6fe2-441c-8c8c-71466251a162>2007-11-18 02:09:36 +0000
commit7317b8cad72b39bf06d1ce76e20a92c7ef157ba6 (patch)
treefb3fd13ec4b274803c77080411f03e46b0282d59
parent9596ebb7012a19a45ca036a200acd617a93a321b (diff)
downloadfocaccia-qemu-7317b8cad72b39bf06d1ce76e20a92c7ef157ba6.tar.gz
focaccia-qemu-7317b8cad72b39bf06d1ce76e20a92c7ef157ba6.zip
Fix a && -> & typo.
Catch wrong/unknown NOR flash command sequences, by Thorsten Zitterell.


git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@3684 c046a42c-6fe2-441c-8c8c-71466251a162
-rw-r--r--hw/pflash_cfi01.c7
-rw-r--r--vl.c2
2 files changed, 7 insertions, 2 deletions
diff --git a/hw/pflash_cfi01.c b/hw/pflash_cfi01.c
index 065dd2c808..f50d7bae77 100644
--- a/hw/pflash_cfi01.c
+++ b/hw/pflash_cfi01.c
@@ -353,6 +353,8 @@ static void pflash_write (pflash_t *pfl, target_ulong offset, uint32_t value,
 
             pfl->counter--;
             break;
+        default:
+            goto error_flash;
         }
         return;
     case 3: /* Confirm mode */
@@ -361,11 +363,14 @@ static void pflash_write (pflash_t *pfl, target_ulong offset, uint32_t value,
             if (cmd == 0xd0) {
                 pfl->wcycle = 0;
                 pfl->status |= 0x80;
-                break;
             } else {
                 DPRINTF("%s: unknown command for \"write block\"\n", __func__);
                 PFLASH_BUG("Write block confirm");
+                goto reset_flash;
             }
+            break;
+        default:
+            goto error_flash;
         }
         return;
     default:
diff --git a/vl.c b/vl.c
index b1e0cc725e..e54afbfb24 100644
--- a/vl.c
+++ b/vl.c
@@ -8269,7 +8269,7 @@ int main(int argc, char **argv)
         kqemu_allowed = 0;
 #endif
     linux_boot = (kernel_filename != NULL);
-    net_boot = (boot_devices_bitmap >> ('n' - 'a')) && 0xF;
+    net_boot = (boot_devices_bitmap >> ('n' - 'a')) & 0xF;
 
     /* XXX: this should not be: some embedded targets just have flash */
     if (!linux_boot && net_boot == 0 &&