Permalink
Browse files

Use a private definition of osockaddr rather then relying on type

namespace polution in sys/socket.h.

Also remove support for operation on 4.3BSD.

PR:		224529
Differential Revision:	https://reviews.freebsd.org/D14505
  • Loading branch information...
brooksdavis committed Jan 18, 2019
1 parent d06a697 commit 3d6c5e19e8b77e8e6a3b3327eae9cf46400946ee
Showing with 16 additions and 14 deletions.
  1. +12 −3 include/protocols/talkd.h
  2. +2 −1 libexec/talkd/talkd.c
  3. +1 −5 usr.bin/talk/invite.c
  4. +1 −5 usr.bin/talk/look_up.c
@@ -54,6 +54,15 @@
* stream connection through which the conversation takes place.
*/

/*
* The talk protocol embeds a 4.3BSD sockaddr. Define our own version
* rather then relying on namespace polution in kernel headers.
*/
struct tsockaddr {
unsigned short sa_family;
char sa_data[14];
};

/*
* Client->server request message format.
*/
@@ -63,8 +72,8 @@ typedef struct {
u_char answer; /* not used */
u_char pad;
u_int32_t id_num; /* message id */
struct osockaddr addr; /* old (4.3) style */
struct osockaddr ctl_addr; /* old (4.3) style */
struct tsockaddr addr; /* old (4.3) style */
struct tsockaddr ctl_addr; /* old (4.3) style */
int32_t pid; /* caller's process id */
#define NAME_SIZE 12
char l_name[NAME_SIZE];/* caller's name */
@@ -82,7 +91,7 @@ typedef struct {
u_char answer; /* respose to request message, see below */
u_char pad;
u_int32_t id_num; /* message id */
struct osockaddr addr; /* address for establishing conversation */
struct tsockaddr addr; /* address for establishing conversation */
} CTL_RESPONSE;

#define TALK_VERSION 1 /* protocol version */
@@ -114,7 +114,8 @@ main(int argc, char *argv[])
continue;
}
lastmsgtime = time(0);
(void)memcpy(&ctl_addr, &mp->ctl_addr, sizeof(ctl_addr));
(void)memcpy(&ctl_addr.sa_data, &mp->ctl_addr.sa_data,
sizeof(ctl_addr.sa_data));
ctl_addr.sa_family = ntohs(mp->ctl_addr.sa_family);
ctl_addr.sa_len = sizeof(ctl_addr);
process_request(mp, &response);
@@ -77,13 +77,9 @@ invite_remote(void)
itimer.it_interval = itimer.it_value;
if (listen(sockt, 5) != 0)
p_error("Error on attempt to listen for caller");
#ifdef MSG_EOR
/* copy new style sockaddr to old, swap family (short in old) */
msg.addr = *(struct osockaddr *)&my_addr; /* XXX new to old style*/
msg.addr = *(struct tsockaddr *)&my_addr;
msg.addr.sa_family = htons(my_addr.sin_family);
#else
msg.addr = *(struct sockaddr *)&my_addr;
#endif
msg.id_num = htonl(-1); /* an impossible id_num */
invitation_waiting = 1;
announce_invite();
@@ -59,13 +59,9 @@ check_local(void)
struct sockaddr addr;

/* the rest of msg was set up in get_names */
#ifdef MSG_EOR
/* copy new style sockaddr to old, swap family (short in old) */
msg.ctl_addr = *(struct osockaddr *)&ctl_addr;
msg.ctl_addr = *(struct tsockaddr *)&ctl_addr;
msg.ctl_addr.sa_family = htons(ctl_addr.sin_family);
#else
msg.ctl_addr = *(struct sockaddr *)&ctl_addr;
#endif
/* must be initiating a talk */
if (!look_for_invite(rp))
return (0);

0 comments on commit 3d6c5e1

Please sign in to comment.