On 08.03.2017 16:09, Peter Meerwald-Stadler wrote: > some more Coverity fixes and cleanups > also addresses Hajime Fujita's earlier review comments > > Peter Meerwald-Stadler (8): > raop: Fix potential dereference after NULL check > raop: Fix potential NULL dereference > raop: Log if pa_atoi() fails, latency is not used anyway > raop: Error out on parsing server port component > core: Assert return value of pa_shared_set/_remove() in dbus-shared > core: Document behaviour of pa_shared_remove() in case name does not > exist > oss: Fix dead code > pulse: Explicitly ignore pa_mainloop_run() return value in thread > function > > src/modules/oss/module-oss.c | 2 +- > src/modules/raop/raop-client.c | 50 +++++++++++++++++++++++++----------------- > src/modules/raop/raop-sink.c | 10 +++++---- > src/pulse/thread-mainloop.c | 2 +- > src/pulsecore/dbus-shared.c | 4 ++-- > src/pulsecore/shared.c | 2 +- > src/pulsecore/shared.h | 4 +++- > 7 files changed, 44 insertions(+), 30 deletions(-) > Look good to me, except where noted.
