From b4f8d31f3bd25dce1e3ab50de7ed7aa757e0ed80 Mon Sep 17 00:00:00 2001
From: Christopher Faylor <me@cgf.cx>
Date: Sun, 12 May 2002 04:43:29 +0000
Subject: [PATCH] * syscalls.cc (_read): Change error to EBADF if attempt to
 write to a non-writable fd.

---
 winsup/cygwin/ChangeLog   | 5 +++++
 winsup/cygwin/syscalls.cc | 8 ++++++++
 2 files changed, 13 insertions(+)

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 09ba3260a..b7940bce2 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,8 @@
+2002-05-12  Christopher Faylor  <cgf@redhat.com>
+
+	* syscalls.cc (_read): Change error to EBADF if attempt to write to a
+	non-writable fd.
+
 2002-05-11  Christopher Faylor  <cgf@redhat.com>
 
 	* fhandler.h (executable_states): For now, make dont_care_if_executable
diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc
index 0be393a46..46ea12fd1 100644
--- a/winsup/cygwin/syscalls.cc
+++ b/winsup/cygwin/syscalls.cc
@@ -340,6 +340,14 @@ _read (int fd, void *ptr, size_t len)
 	}
 
     out:
+
+      if (res && get_errno () == EACCES &&
+	  !(cfd->get_flags () & (O_RDONLY | O_RDWR)))
+	{
+	  set_errno (EBADF);
+	  break;
+	}
+
       if (res >= 0 || get_errno () != EINTR || !thisframe.call_signal_handler ())
 	break;
       set_errno (e);