Summary: | X attempts to use shared memory even for ssh-proxied connections | ||||||
---|---|---|---|---|---|---|---|
Product: | xorg | Reporter: | Mark Seaborn <mseaborn> | ||||
Component: | Server/General | Assignee: | Xorg Project Team <xorg-team> | ||||
Status: | RESOLVED FIXED | QA Contact: | Xorg Project Team <xorg-team> | ||||
Severity: | minor | ||||||
Priority: | lowest | CC: | ajax, esigra | ||||
Version: | 7.1 (2006.05) | ||||||
Hardware: | x86 (IA32) | ||||||
OS: | Linux (All) | ||||||
Whiteboard: | |||||||
i915 platform: | i915 features: | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 44202 | ||||||
Attachments: |
|
Description
Mark Seaborn
2007-05-29 06:23:14 UTC
Created attachment 10127 [details] [review] Patch against libXext/src/XShm.c to detect remote connections We've had the attached code in the Solaris libXext for a while, to check if an X connection is local or remote, and then return that MIT-SHM is unavailable to remote - but the ssh proxy would probably confuse this code as well. (And I'm not sure how portable it is.) So I hacked up server-side support for this: http://cvs.fedoraproject.org/viewcvs/*checkout*/rpms/xorg-x11-server/devel/xserver-1.4.99-ssh-isnt-local.patch?rev=1.2 It's not pretty, and it's certainly a linuxism, but it does reject ssh client processes correctly. I think, ideally, we'd just redact MIT-SHM from the extension list for non-local clients (and the DRI extensions too), _and_ fix the non-local extensions to throw BadRequest should a client attempt to guess at the hidden extension anyway. Can we commit this already? (In reply to comment #3) > Can we commit this already? apparently not. i'm sure it'll definitely be done properly in 1.8 though, for serious reals. Ideally the X tunnelling in ssh would intercept the extension requests itself for MIT-SHM and reject them (probably would want to have a blacklist in the ssh config of extensions like this to hide from tunneled clients), but that's something to address with the OpenSSH community. (In reply to comment #4) > apparently not. i'm sure it'll definitely be done properly in 1.8 though, for > serious reals. Or maybe 1.13 if ajax's latest patch gets accepted. We have ssh detection now. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.