From 11b8a2aad4ff3e6d1cfde08ce836a72a05c69a15 Mon Sep 17 00:00:00 2001
From: Wim Taymans <wtaymans@redhat.com>
Date: Mon, 29 Nov 2021 11:23:32 +0100
Subject: [PATCH] buffer: handle the case where only MemPtr is selected

When we need to allocate buffers with shared memory but only MemPtr is
allowed, make sure we change this to MemFd.
---
 src/pipewire/buffers.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/pipewire/buffers.c b/src/pipewire/buffers.c
index f15f75753..e0b74eb25 100644
--- a/src/pipewire/buffers.c
+++ b/src/pipewire/buffers.c
@@ -318,10 +318,12 @@ int pw_buffers_negotiate(struct pw_context *context, uint32_t flags,
 		minsize = 8192;
 		max_buffers = 2;
 	}
-
-	if (types == SPA_ID_INVALID &&
-	    SPA_FLAG_IS_SET(flags, PW_BUFFERS_FLAG_SHARED_MEM))
-		types = 1<<SPA_DATA_MemFd;
+	if (SPA_FLAG_IS_SET(flags, PW_BUFFERS_FLAG_SHARED_MEM)) {
+		if (types != SPA_ID_INVALID)
+			SPA_FLAG_CLEAR(types, 1<<SPA_DATA_MemPtr);
+		if (types == 0 || types == SPA_ID_INVALID)
+			types = 1<<SPA_DATA_MemFd;
+	}
 
 	if (SPA_FLAG_IS_SET(flags, PW_BUFFERS_FLAG_NO_MEM))
 		minsize = 0;