On 24/10/19 17:14, Philippe Mathieu-Daudé wrote: > On 10/24/19 4:03 PM, Paolo Bonzini wrote: >> Reported by Coverity (CID 1406449). >> >> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > > I reviewed this one.
Oops, sorry. I must have lost your R-b when moving the patch from one computer to the other. Paolo >> --- >> audio/paaudio.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/audio/paaudio.c b/audio/paaudio.c >> index df541a7..55a91f8 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); >>