Coverity thinks that sample can be NULL when it's
dereferenced after this line. Adding an assertion doesn't
hurt here (in my opinion), and that should get rid of the
warning.
---
src/modules/dbus/iface-core.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/modules/dbus/iface-core.c b/src/modules/dbus/iface-core.c
index 58abcb9..97a46a5 100644
--- a/src/modules/dbus/iface-core.c
+++ b/src/modules/dbus/iface-core.c
@@ -1401,7 +1401,7 @@ static void handle_upload_sample(DBusConnection *conn,
DBusMessage *msg, void *u
goto finish;
}
- sample = pa_idxset_get_by_index(c->core->scache, idx);
+ pa_assert_se(sample = pa_idxset_get_by_index(c->core->scache, idx));
if (n_volume_entries > 0) {
sample->volume.channels = n_channels;
--
1.7.8
_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
