Re: [Qemu-devel] [PATCH] bt-host: add missing break statement

[Stefan Weil]

Am 12.01.2012 15:17, schrieb Stefan Hajnoczi:

The switch statement in bt_host_read() is missing a break in one case.
Andrzej Zaborowski  confirmed that this is
not an intentional fall-through.

Signed-off-by: Stefan Hajnoczi 
---
bt-host.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/bt-host.c b/bt-host.c
index df5b7cd..0d3ad28 100644
--- a/bt-host.c
+++ b/bt-host.c
@@ -130,6 +130,7 @@ static void bt_host_read(void *opaque)
pktlen = MIN(pkt[2] + 3, s->len);
s->len -= pktlen;
pkt += pktlen;
+ break;

default:
bad_pkt:


Reviewed-by: Stefan Weil 

[Qemu-devel] [PATCH] bt-host: add missing break statement

[Stefan Hajnoczi]

The switch statement in bt_host_read() is missing a break in one case.
Andrzej Zaborowski  confirmed that this is
not an intentional fall-through.

Signed-off-by: Stefan Hajnoczi 
---
 bt-host.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/bt-host.c b/bt-host.c
index df5b7cd..0d3ad28 100644
--- a/bt-host.c
+++ b/bt-host.c
@@ -130,6 +130,7 @@ static void bt_host_read(void *opaque)
 pktlen = MIN(pkt[2] + 3, s->len);
 s->len -= pktlen;
 pkt += pktlen;
+break;
 
 default:
 bad_pkt:
-- 
1.7.7.3