diff options
author | snj <snj@pkgsrc.org> | 2009-05-19 19:41:18 +0000 |
---|---|---|
committer | snj <snj@pkgsrc.org> | 2009-05-19 19:41:18 +0000 |
commit | e9a6d9f2273bf89b6311d79cd9e666328e14bd47 (patch) | |
tree | 8c0834812bfac3c0b27d9ad0c17684824621e360 /x11/rxvt-unicode/patches | |
parent | 65008d016793dc0204112111c0936194bb66dd2b (diff) | |
download | pkgsrc-e9a6d9f2273bf89b6311d79cd9e666328e14bd47.tar.gz |
Add the upstream version of a patch submitted by Talor R Campbell in
PR pkg/38988 to fix a problem with socket control message ancillary data
alignment.
Bump PKGREVISION to 1.
Diffstat (limited to 'x11/rxvt-unicode/patches')
-rw-r--r-- | x11/rxvt-unicode/patches/patch-ad | 120 |
1 files changed, 120 insertions, 0 deletions
diff --git a/x11/rxvt-unicode/patches/patch-ad b/x11/rxvt-unicode/patches/patch-ad new file mode 100644 index 00000000000..8fb20b65da9 --- /dev/null +++ b/x11/rxvt-unicode/patches/patch-ad @@ -0,0 +1,120 @@ +$NetBSD: patch-ad,v 1.1 2009/05/19 19:41:19 snj Exp $ + +--- src/fdpass.C.orig 2008-05-05 09:51:47.000000000 -0700 ++++ src/fdpass.C 2009-05-19 12:28:03.000000000 -0700 +@@ -26,6 +26,7 @@ + #include "../config.h" + + #include <cstddef> // needed by broken bsds for NULL used in sys/uio.h ++#include <cstdlib> + + #include <sys/types.h> + #include <sys/uio.h> +@@ -33,16 +34,26 @@ + + #include "libptytty.h" + +-#ifndef CMSG_LEN // CMSG_SPACE && CMSG_LEN are rfc2292 extensions to unix ++// CMSG_SPACE & CMSG_LEN are rfc2292 extensions to unix ++#ifndef CMSG_SPACE ++# define CMSG_SPACE(len) (sizeof (cmsghdr) + len) ++#endif ++ ++#ifndef CMSG_LEN + # define CMSG_LEN(len) (sizeof (cmsghdr) + len) + #endif + + bool + ptytty::send_fd (int socket, int fd) + { ++ void *buf = malloc (CMSG_SPACE (sizeof (int))); ++ ++ if (!buf) ++ return 0; ++ + msghdr msg; + iovec iov; +- char buf [CMSG_LEN (sizeof (int))]; ++ cmsghdr *cmsg; + char data = 0; + + iov.iov_base = &data; +@@ -52,27 +63,33 @@ ptytty::send_fd (int socket, int fd) + msg.msg_namelen = 0; + msg.msg_iov = &iov; + msg.msg_iovlen = 1; +- msg.msg_control = (void *)buf; +- msg.msg_controllen = sizeof buf; ++ msg.msg_control = buf; ++ msg.msg_controllen = CMSG_SPACE (sizeof (int)); + +- cmsghdr *cmsg = CMSG_FIRSTHDR (&msg); ++ cmsg = CMSG_FIRSTHDR (&msg); + cmsg->cmsg_level = SOL_SOCKET; + cmsg->cmsg_type = SCM_RIGHTS; + cmsg->cmsg_len = CMSG_LEN (sizeof (int)); + + *(int *)CMSG_DATA (cmsg) = fd; + +- msg.msg_controllen = cmsg->cmsg_len; ++ ssize_t result = sendmsg (socket, &msg, 0); ++ ++ free (buf); + +- return sendmsg (socket, &msg, 0) >= 0; ++ return result >= 0; + } + + int + ptytty::recv_fd (int socket) + { ++ void *buf = malloc (CMSG_SPACE (sizeof (int))); ++ ++ if (!buf) ++ return -1; ++ + msghdr msg; + iovec iov; +- char buf [CMSG_LEN (sizeof (int))]; /* ancillary data buffer */ + char data = 1; + + iov.iov_base = &data; +@@ -83,23 +100,24 @@ ptytty::recv_fd (int socket) + msg.msg_iov = &iov; + msg.msg_iovlen = 1; + msg.msg_control = buf; +- msg.msg_controllen = sizeof buf; ++ msg.msg_controllen = CMSG_SPACE (sizeof (int)); + +- cmsghdr *cmsg = CMSG_FIRSTHDR (&msg); +- cmsg->cmsg_level = SOL_SOCKET; +- cmsg->cmsg_type = SCM_RIGHTS; +- cmsg->cmsg_len = CMSG_LEN (sizeof (int)); ++ int fd = -1; + +- msg.msg_controllen = cmsg->cmsg_len; ++ if (recvmsg (socket, &msg, 0) > 0 ++ && data == 0 ++ && msg.msg_controllen >= CMSG_SPACE (sizeof (int))) ++ { ++ cmsghdr *cmsg = CMSG_FIRSTHDR (&msg); + +- if (recvmsg (socket, &msg, 0) <= 0 +- || data != 0 +- || msg.msg_controllen < CMSG_LEN (sizeof (int)) +- || cmsg->cmsg_level != SOL_SOCKET +- || cmsg->cmsg_type != SCM_RIGHTS +- || cmsg->cmsg_len < CMSG_LEN (sizeof (int))) +- return -1; ++ if (cmsg->cmsg_level == SOL_SOCKET ++ && cmsg->cmsg_type == SCM_RIGHTS ++ && cmsg->cmsg_len >= CMSG_LEN (sizeof (int))) ++ fd = *(int *)CMSG_DATA (cmsg); ++ } ++ ++ free (buf); + +- return *(int *)CMSG_DATA (cmsg); ++ return fd; + } + |