From 603c2ea98cf0934f1ad0f9511dae31cfb372e9f2 Mon Sep 17 00:00:00 2001
From: Patrick Williams <pdwilliams@google.com>
Date: Mon, 28 Oct 2024 15:25:47 -0500
Subject: Add unit tests to check that BufferReleaseChannel is unidirectional

Bug: 294133380
Flag: com.android.graphics.libgui.flags.buffer_release_channel
Test: BufferReleaseChannelTest
Change-Id: Iae20dc9e24826fbfd372717d64081a40c7ab1bab
---
 libs/gui/BufferReleaseChannel.cpp | 7 -------
 1 file changed, 7 deletions(-)

(limited to 'libs/gui/BufferReleaseChannel.cpp')

diff --git a/libs/gui/BufferReleaseChannel.cpp b/libs/gui/BufferReleaseChannel.cpp
index eb50684412..e9cb013baf 100644
--- a/libs/gui/BufferReleaseChannel.cpp
+++ b/libs/gui/BufferReleaseChannel.cpp
@@ -339,13 +339,6 @@ status_t BufferReleaseChannel::open(std::string name,
         return -errno;
     }
 
-    // Make the producer write-only
-    if (shutdown(producerFd.get(), SHUT_RD) == -1) {
-        ALOGE("[%s] Failed to shutdown reading on producer socket. errno=%d message='%s'",
-              name.c_str(), errno, strerror(errno));
-        return -errno;
-    }
-
     outConsumer = std::make_unique<ConsumerEndpoint>(name, std::move(consumerFd));
     outProducer = std::make_shared<ProducerEndpoint>(std::move(name), std::move(producerFd));
     return STATUS_OK;
-- 
cgit v1.2.3-59-g8ed1b