[SECNET PATCH 03/12] site: Rename setup_priority to our_name_later

Ian Jackson ijackson at chiark.greenend.org.uk
Wed May 15 23:13:28 BST 2019


This is also used for setting the transform direction - ie,
distinguishing us and them.  We are going to make the priority system
more subtle, don't want to mess with that bit.

No functional change.

Signed-off-by: Ian Jackson <ijackson at chiark.greenend.org.uk>
---
 site.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/site.c b/site.c
index 67edb2f..2a62293 100644
--- a/site.c
+++ b/site.c
@@ -328,8 +328,7 @@ struct site {
 				      after this time, initiate a new
 				      key exchange */
 
-    bool_t setup_priority; /* Do we have precedence if both sites emit
-			      message 1 simultaneously? */
+    bool_t our_name_later; /* our name > peer name */
     uint32_t log_events;
 
 /* runtime information */
@@ -554,7 +553,7 @@ static void set_new_transform(struct site *st, char *pk)
     struct transform_if *generator=st->chosen_transform;
     struct transform_inst_if *generated=generator->create(generator->st);
     generated->setkey(generated->st,st->sharedsecret,
-		      st->sharedsecretlen,st->setup_priority);
+		      st->sharedsecretlen,st->our_name_later);
     dispose_transform(&st->new_transform);
     st->new_transform=generated;
 
@@ -1783,7 +1782,7 @@ static bool_t site_incoming(void *sst, struct buffer_if *buf,
 	    /* We've just sent a message 1! They may have crossed on
 	       the wire. If we have priority then we ignore the
 	       incoming one, otherwise we process it as usual. */
-	    if (st->setup_priority) {
+	    if (st->our_name_later) {
 		BUF_FREE(buf);
 		slog(st,LOG_DUMP,"crossed msg1s; we are higher "
 		     "priority => ignore incoming msg1");
@@ -2115,7 +2114,7 @@ static list_t *site_apply(closure_t *self, struct cloc loc, dict_t *context,
     /* The information we expect to see in incoming messages of type 1 */
     /* fixme: lots of unchecked overflows here, but the results are only
        corrupted packets rather than undefined behaviour */
-    st->setup_priority=(strcmp(st->localname,st->remotename)>0);
+    st->our_name_later=(strcmp(st->localname,st->remotename)>0);
 
     buffer_new(&st->buffer,SETUP_BUFFER_LEN);
 
-- 
2.11.0




More information about the sgo-software-discuss mailing list