On Tue, Jan 14, 2020 at 06:29:52PM -0500, Thomas Dickey wrote: > On Tue, Jan 14, 2020 at 10:23:15PM +0100, Matthieu Herrb wrote: > > This code has a number of issues: > > - It doesn't support IPv6 https://bugs.freedesktop.org/show_bug.cgi?id=7611 > > - The IP address is set by the selection owner so it doesn't bring any > > security for anyone querying the selection > > - The extra DNS query that it does is problematic in some sandboxed > > environments (like OpenBSD's pledge > > https://marc.info/?l=openbsd-bugs&m=157842725819911&w=2) > > > > Signed-off-by: Matthieu Herrb <matth...@herrb.eu> > > --- > > src/CvtStdSel.c | 30 ++---------------------------- > > 1 file changed, 2 insertions(+), 28 deletions(-) > > The documentation should be updated as well.
Oh yes, you're right. I think the patch below is enough. Or should we mention explicitely that IP_ADDRESSS is now deprecated ? From eb1b1c9fd4336bd2b92e8bbb174f8d829a9dac89 Mon Sep 17 00:00:00 2001 From: Matthieu Herrb <matth...@herrb.eu> Date: Wed, 15 Jan 2020 07:55:53 +0100 Subject: [PATCH] doc: IP_ADDRESS is not converted by XmuConvertStandardSelection anymore. Signed-off-by: Matthieu Herrb <matth...@herrb.eu> --- doc/Xmu.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/Xmu.xml b/doc/Xmu.xml index 2d363a9..fdd60e0 100644 --- a/doc/Xmu.xml +++ b/doc/Xmu.xml @@ -1442,7 +1442,7 @@ returns the size in bits of the elements</para> </variablelist> This function converts the following standard selections: CLASS, -CLIENT_WINDOW, DECNET_ADDRESS, HOSTNAME, IP_ADDRESS, NAME, OWNER_OS, +CLIENT_WINDOW, DECNET_ADDRESS, HOSTNAME, NAME, OWNER_OS, TARGETS, TIMESTAMP, and USER. It returns <symbol>True</symbol> if the conversion was successful, else it returns -- 2.24.1 -- Matthieu Herrb
signature.asc
Description: PGP signature
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel