[pulseaudio-discuss] [PATCH v2 00/12] patches addressing some Coverity issues

Arun Raghavan arun at accosted.net
Tue Sep 15 20:55:42 PDT 2015


On Wed, 2015-09-16 at 00:01 +0200, Peter Meerwald wrote:
> nota bene: put the following in user_nodefs.h in the cov
> -analysis/config
> directory to address pa_assert_se() false alarms:
> 
>  #nodef pa_assert_se(x) do { int y = x ; if (!(y))
> __coverity_panic__(); } while (0)
> 
> fix assorted issues, more to come...
> 
> v2 fixes the incomplete 'modules: Fix entry leak in module-card
> -restore' and drops
> 'modules: Fix dead code in oss-util' (thanks, Arun)
> 
> v2 adds patch 8 to 12
> 
> I intend to apply 
> 'daemon: No need to check optarg, -p requires argument'
> 'modules: Fix entry leak in module-card-restore'
> 'core: Add missing return on protocol error'
> to master soonish

Ack.

> the remaining patches will go to next after review

Also ack'ed.

> Peter Meerwald (12):
>   modules: Fix unchecked return in module-card-restore
>   core: Assert that memblockq's base is != 0
>   tests: Assert fillrate > 0 in alsa-time-test
>   daemon: No need to check optarg, -p requires argument
>   modules: Fix entry leak in module-card-restore
>   modules: Cleanup get_sinks() in module-equalizer-sink
>   core: Add missing return on protocol error
>   modules: Use pa_assert_se() to check return value of
>     dbus_message_iter_close_container()
>   modules: Check pa_threaded_mainloop_start() return value
>   tests: Check pa_rtpoll_run() return value
>   stream: Check pa_tagstruct_get_format_info() retval in
>     pa_create_stream_callback()
>   alsa: Check pa_modargs_get_value_boolean() retval for use_ucm

-- Arun


More information about the pulseaudio-discuss mailing list