krb5 commit: Remove is_udp field in sendto_kdc.c conn_state
Greg Hudson
ghudson at MIT.EDU
Tue Jul 30 12:47:15 EDT 2013
https://github.com/krb5/krb5/commit/ee222531c802531a1ac0a74c933dfdfa48e8ed9e
commit ee222531c802531a1ac0a74c933dfdfa48e8ed9e
Author: Robbie Harwood (frozencemetery) <rharwood at club.cc.cmu.edu>
Date: Mon Jul 22 16:11:41 2013 -0400
Remove is_udp field in sendto_kdc.c conn_state
This field is redundant with addr.type.
[ghudson at mit.edu: removed extraneous changes; clarified commit
message]
src/lib/krb5/os/sendto_kdc.c | 6 +-----
1 files changed, 1 insertions(+), 5 deletions(-)
diff --git a/src/lib/krb5/os/sendto_kdc.c b/src/lib/krb5/os/sendto_kdc.c
index 3c31d9f..ecea68f 100644
--- a/src/lib/krb5/os/sendto_kdc.c
+++ b/src/lib/krb5/os/sendto_kdc.c
@@ -89,7 +89,6 @@ struct incoming_krb5_message {
struct conn_state {
SOCKET fd;
enum conn_states state;
- unsigned int is_udp : 1;
int (*service)(krb5_context context, struct conn_state *,
struct select_state *, int);
struct remote_address addr;
@@ -432,8 +431,7 @@ set_conn_state_msg_length (struct conn_state *state, const krb5_data *message)
if (!message || message->length == 0)
return;
- if (!state->is_udp) {
-
+ if (state->addr.type == SOCK_STREAM) {
store_32_be(message->length, state->x.out.msg_len_buf);
SG_SET(&state->x.out.sgbuf[0], state->x.out.msg_len_buf, 4);
SG_SET(&state->x.out.sgbuf[1], message->data, message->length);
@@ -473,7 +471,6 @@ add_connection(struct conn_state **conns, struct addrinfo *ai,
state->x.out.sg_count = 2;
*/
- state->is_udp = 0;
state->service = service_tcp_fd;
set_conn_state_msg_length (state, message);
} else {
@@ -483,7 +480,6 @@ add_connection(struct conn_state **conns, struct addrinfo *ai,
state->x.out.sg_count = 1;
*/
- state->is_udp = 1;
state->service = service_udp_fd;
set_conn_state_msg_length (state, message);
More information about the cvs-krb5
mailing list