[pulseaudio-discuss] [PATCH v2 00/12] patches addressing some Coverity issues
Peter Meerwald
pmeerw at pmeerw.net
Tue Sep 15 15:01:38 PDT 2015
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
the remaining patches will go to next after review
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
src/daemon/cmdline.c | 2 +-
src/modules/alsa/module-alsa-card.c | 6 +++++-
src/modules/module-card-restore.c | 8 ++++++--
src/modules/module-equalizer-sink.c | 14 +++++++-------
src/modules/module-ladspa-sink.c | 2 +-
src/modules/module-zeroconf-publish.c | 3 ++-
src/pulse/stream.c | 8 +++-----
src/pulsecore/memblockq.c | 2 +-
src/pulsecore/protocol-native.c | 4 +++-
src/tests/alsa-time-test.c | 1 +
src/tests/rtpoll-test.c | 4 ++--
11 files changed, 32 insertions(+), 22 deletions(-)
--
1.9.1
More information about the pulseaudio-discuss
mailing list