[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PULL 04/14] audio: -audiodev command line option basic i
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PULL 04/14] audio: -audiodev command line option basic implementation |
Date: |
Thu, 14 Mar 2019 09:46:18 +0000 |
On Tue, 12 Mar 2019 at 07:13, Gerd Hoffmann <address@hidden> wrote:
>
> From: Kővágó, Zoltán <address@hidden>
>
> Audio drivers now get an Audiodev * as config paramters, instead of the
> global audio_option structs. There is some code in audio/audio_legacy.c
> that converts the old environment variables to audiodev options (this
> way backends do not have to worry about legacy options). It also
> contains a replacement of -audio-help, which prints out the equivalent
> -audiodev based config of the currently specified environment variables.
Hi; Coverity complains (CID 1399706) about this, which isn't
a change in this patch as such, but the code change has
probably caused it to reanalyze:
>
> if (!done) {
> driver = audio_driver_lookup("none");
> - done = !audio_driver_init(s, driver, false);
> + done = !audio_driver_init(s, driver, false, dev);
Everywhere else we call audio_driver_lookup() we check
whether the return value is NULL before using it,
but here we don't. I guess this is a false positive
because the "none" driver must always exist ?
If so, I can just silence the warning in the coverity UI.
thanks
-- PMM
- [Qemu-arm] [PULL 07/14] dsoundaudio: port to -audiodev config, (continued)
- [Qemu-arm] [PULL 07/14] dsoundaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 13/14] wavaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 05/14] alsaaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 11/14] sdlaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 09/14] ossaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 10/14] paaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 08/14] noaudio: port to -audiodev config, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 01/14] qapi: qapi for audio backends, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 14/14] audio: -audiodev command line option: cleanup, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 04/14] audio: -audiodev command line option basic implementation, Gerd Hoffmann, 2019/03/12
- Re: [Qemu-arm] [PULL 04/14] audio: -audiodev command line option basic implementation,
Peter Maydell <=
- [Qemu-arm] [PULL 02/14] audio: use qapi AudioFormat instead of audfmt_e, Gerd Hoffmann, 2019/03/12
- [Qemu-arm] [PULL 03/14] audio: -audiodev command line option: documentation, Gerd Hoffmann, 2019/03/12
- Re: [Qemu-arm] [PULL 00/14] Audio 20190312 patches, Peter Maydell, 2019/03/12