Re: [PATCH] audio: fix missing break

[Philippe Mathieu-Daudé]

On 10/23/19 10:24 AM, Paolo Bonzini wrote:
> Reported by Coverity (CID 1406449).

Gerd sent the same patch earlier:
https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg05923.html

But your description is better.
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>

> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>   audio/paaudio.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/audio/paaudio.c b/audio/paaudio.c
> index df541a72d3..55a91f8980 100644
> --- a/audio/paaudio.c
> +++ b/audio/paaudio.c
> @@ -385,6 +385,7 @@ static pa_stream *qpa_simple_new (
>           map.map[5] = PA_CHANNEL_POSITION_REAR_RIGHT;
>           map.map[6] = PA_CHANNEL_POSITION_SIDE_LEFT;
>           map.map[7] = PA_CHANNEL_POSITION_SIDE_RIGHT;
> +        break;
>   
>       default:
>           dolog("Internal error: unsupported channel count %d\n", ss->channels);