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