[pulseaudio-commits] src/pulsecore

[David Henningsson]

 src/pulsecore/srbchannel.c |    3 +++
 1 file changed, 3 insertions(+)

New commits:
commit e175796fa5a799dfdb71c928fedada5fac315e9c
Author: David Henningsson <david.henningsson at canonical.com>
Date:   Thu Dec 4 22:06:30 2014 +0100

    srbchannel: Check return value of pa_memblock_new_pool
    
    In case PA_MEMPOOL_DISABLE is set, pa_memblock_new_pool can return
    NULL. It does not make sense to set up a srbchannel without a shared
    memory pool, so just fail in this case.
    
    Reported-by: Alexander Patrakov <patrakov at gmail.com>
    Signed-off-by: David Henningsson <david.henningsson at canonical.com>

diff --git a/src/pulsecore/srbchannel.c b/src/pulsecore/srbchannel.c
index a0f916e..8872a89 100644
--- a/src/pulsecore/srbchannel.c
+++ b/src/pulsecore/srbchannel.c
@@ -231,6 +231,9 @@ pa_srbchannel* pa_srbchannel_new(pa_mainloop_api *m, pa_mempool *p) {
     pa_srbchannel* sr = pa_xmalloc0(sizeof(pa_srbchannel));
     sr->mainloop = m;
     sr->memblock = pa_memblock_new_pool(p, -1);
+    if (!sr->memblock)
+        goto fail;
+
     srh = pa_memblock_acquire(sr->memblock);
     pa_zero(*srh);