diff options
author | Peter Meerwald-Stadler <pmeerw@pmeerw.net> | 2017-03-07 14:43:48 +0100 |
---|---|---|
committer | Peter Meerwald-Stadler <pmeerw@pmeerw.net> | 2017-03-09 13:51:51 +0100 |
commit | a8ce3ef844f460b1769e6472694ca4cf049dfd55 (patch) | |
tree | efd3159bb099300bbe6d8b97ba905d92673fc35f /src/modules/raop | |
parent | bfe0e6366d091562972a89d5e343f550509291bd (diff) | |
download | pulseaudio-a8ce3ef844f460b1769e6472694ca4cf049dfd55.tar.gz |
raop: Fix potential dereference after NULL check
Coverity ID: #1398157
Signed-off-by: Peter Meerwald-Stadler <pmeerw@pmeerw.net>
Diffstat (limited to 'src/modules/raop')
-rw-r--r-- | src/modules/raop/raop-sink.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/modules/raop/raop-sink.c b/src/modules/raop/raop-sink.c index c5ff8b9c0..4ca625f0c 100644 --- a/src/modules/raop/raop-sink.c +++ b/src/modules/raop/raop-sink.c @@ -246,10 +246,12 @@ static int sink_process_msg(pa_msgobject *o, int code, void *data, int64_t offse if (u->rtpoll_item) { pollfd = pa_rtpoll_item_get_pollfd(u->rtpoll_item, &nbfds); - for (i = 0; i < nbfds; i++) { - if (pollfd && pollfd->fd >= 0) - pa_close(pollfd->fd); - pollfd++; + if (pollfd) { + for (i = 0; i < nbfds; i++) { + if (pollfd->fd >= 0) + pa_close(pollfd->fd); + pollfd++; + } } pa_rtpoll_item_free(u->rtpoll_item); u->rtpoll_item = NULL; |