[PATCH 1/3 xserver] xace: Add new hook to override Xtrans Read and Write invocations

Andrew Eikum aeikum at codeweavers.com
Fri Jan 18 07:43:00 PST 2013


This hook is useful for environments where the security extension might
need to inject data into the stream or process out-of-band information
from clients. Between this and the following patch, I believe I have
added relevant hooks to all accept/read/write/close/disconnect calls in
the tree.

I only hook those five calls. I don't add hooks for other auxiliary
Xtrans calls such as XTransSetOption. The idea is that a security
extension would copy the existing Xtrans implementation and add
modifications. I didn't intend to allow security extensions to create an
entirely "hot-pluggable" Xtrans in a security extension.

Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
---

This is a resend of my three patches, rebased onto the latest
development branch. The first attempt[1] was over the holiday season,
so may have just been dropped on the floor.

Looking at the log for <xserver:Xext/xace.c>, it looks like XACE
doesn't see much active development these days, so perhaps there's no
one in particular to review these patches. I've CCd Eamon Walsh on the
series again, but otherwise what can I do to get these into the X.org
tree?

Thanks,
Andrew

[1]
http://lists.freedesktop.org/archives/xorg-devel/2012-December/034872.html

 Xext/xace.c     | 17 ++++++++++++++++-
 Xext/xace.h     |  8 +++++++-
 Xext/xacestr.h  | 12 ++++++++++++
 os/connection.c | 12 +++++++++---
 os/io.c         | 30 +++++++++++++++++++++++++-----
 5 files changed, 69 insertions(+), 10 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-xace-Add-new-hook-to-override-Xtrans-Read-and-Write-.patch
Type: text/x-patch
Size: 6508 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20130118/b4e35c2c/attachment-0001.bin>


More information about the xorg-devel mailing list