[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