diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | monitor_fdpass.c | 16 |
2 files changed, 16 insertions, 6 deletions
@@ -30,6 +30,10 @@ | |||
30 | add an extension method "posix-rename@openssh.com" to perform POSIX atomic | 30 | add an extension method "posix-rename@openssh.com" to perform POSIX atomic |
31 | rename() operations. based on patch from miklos AT szeredi.hu in bz#1400; | 31 | rename() operations. based on patch from miklos AT szeredi.hu in bz#1400; |
32 | ok dtucker@ markus@ | 32 | ok dtucker@ markus@ |
33 | - deraadt@cvs.openbsd.org 2008/03/02 18:19:35 | ||
34 | [monitor_fdpass.c] | ||
35 | use a union to ensure alignment of the cmsg (pay attention: various other | ||
36 | parts of the tree need this treatment too); ok djm | ||
33 | 37 | ||
34 | 20080302 | 38 | 20080302 |
35 | - (dtucker) [configure.ac] FreeBSD's glob() doesn't behave the way we expect | 39 | - (dtucker) [configure.ac] FreeBSD's glob() doesn't behave the way we expect |
@@ -3690,4 +3694,4 @@ | |||
3690 | OpenServer 6 and add osr5bigcrypt support so when someone migrates | 3694 | OpenServer 6 and add osr5bigcrypt support so when someone migrates |
3691 | passwords between UnixWare and OpenServer they will still work. OK dtucker@ | 3695 | passwords between UnixWare and OpenServer they will still work. OK dtucker@ |
3692 | 3696 | ||
3693 | $Id: ChangeLog,v 1.4857 2008/03/07 07:33:53 djm Exp $ | 3697 | $Id: ChangeLog,v 1.4858 2008/03/07 07:35:26 djm Exp $ |
diff --git a/monitor_fdpass.c b/monitor_fdpass.c index a572302e8..fb00ab7ab 100644 --- a/monitor_fdpass.c +++ b/monitor_fdpass.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: monitor_fdpass.c,v 1.13 2007/09/04 03:21:03 djm Exp $ */ | 1 | /* $OpenBSD: monitor_fdpass.c,v 1.14 2008/03/02 18:19:35 deraadt Exp $ */ |
2 | /* | 2 | /* |
3 | * Copyright 2001 Niels Provos <provos@citi.umich.edu> | 3 | * Copyright 2001 Niels Provos <provos@citi.umich.edu> |
4 | * All rights reserved. | 4 | * All rights reserved. |
@@ -49,7 +49,10 @@ mm_send_fd(int sock, int fd) | |||
49 | char ch = '\0'; | 49 | char ch = '\0'; |
50 | ssize_t n; | 50 | ssize_t n; |
51 | #ifndef HAVE_ACCRIGHTS_IN_MSGHDR | 51 | #ifndef HAVE_ACCRIGHTS_IN_MSGHDR |
52 | char tmp[CMSG_SPACE(sizeof(int))]; | 52 | union { |
53 | struct cmsghdr hdr; | ||
54 | char tmp[CMSG_SPACE(sizeof(int))]; | ||
55 | } tmp; | ||
53 | struct cmsghdr *cmsg; | 56 | struct cmsghdr *cmsg; |
54 | #endif | 57 | #endif |
55 | 58 | ||
@@ -58,7 +61,7 @@ mm_send_fd(int sock, int fd) | |||
58 | msg.msg_accrights = (caddr_t)&fd; | 61 | msg.msg_accrights = (caddr_t)&fd; |
59 | msg.msg_accrightslen = sizeof(fd); | 62 | msg.msg_accrightslen = sizeof(fd); |
60 | #else | 63 | #else |
61 | msg.msg_control = (caddr_t)tmp; | 64 | msg.msg_control = (caddr_t)&tmp; |
62 | msg.msg_controllen = CMSG_LEN(sizeof(int)); | 65 | msg.msg_controllen = CMSG_LEN(sizeof(int)); |
63 | cmsg = CMSG_FIRSTHDR(&msg); | 66 | cmsg = CMSG_FIRSTHDR(&msg); |
64 | cmsg->cmsg_len = CMSG_LEN(sizeof(int)); | 67 | cmsg->cmsg_len = CMSG_LEN(sizeof(int)); |
@@ -100,7 +103,10 @@ mm_receive_fd(int sock) | |||
100 | char ch; | 103 | char ch; |
101 | int fd; | 104 | int fd; |
102 | #ifndef HAVE_ACCRIGHTS_IN_MSGHDR | 105 | #ifndef HAVE_ACCRIGHTS_IN_MSGHDR |
103 | char tmp[CMSG_SPACE(sizeof(int))]; | 106 | union { |
107 | char tmp[CMSG_SPACE(sizeof(int))]; | ||
108 | struct cmsghdr hdr; | ||
109 | } tmp; | ||
104 | struct cmsghdr *cmsg; | 110 | struct cmsghdr *cmsg; |
105 | #endif | 111 | #endif |
106 | 112 | ||
@@ -113,7 +119,7 @@ mm_receive_fd(int sock) | |||
113 | msg.msg_accrights = (caddr_t)&fd; | 119 | msg.msg_accrights = (caddr_t)&fd; |
114 | msg.msg_accrightslen = sizeof(fd); | 120 | msg.msg_accrightslen = sizeof(fd); |
115 | #else | 121 | #else |
116 | msg.msg_control = tmp; | 122 | msg.msg_control = &tmp; |
117 | msg.msg_controllen = sizeof(tmp); | 123 | msg.msg_controllen = sizeof(tmp); |
118 | #endif | 124 | #endif |
119 | 125 | ||