On 08.03.2017 16:09, Peter Meerwald-Stadler wrote:
some more Coverity fixes and cleanups also addresses Hajime Fujita's earlier review commentsPeter 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. _______________________________________________ pulseaudio-discuss mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
