[pulseaudio-discuss] [PATCH 0/8] more Coverity fixes and cleanups
Georg Chini
georg at chini.tk
Thu Mar 9 06:57:36 UTC 2017
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.
More information about the pulseaudio-discuss
mailing list