Fix typo in .clang-tidy

cisco · Jul 19, 2023 · 2245e97 · 2245e97
1 parent f4125c3
commit 2245e97
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 10 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -5,7 +5,7 @@ Checks:            '*,
                     -bugprone-easily-swappable-parameters,
                     -bugprone-exception-escape,
                     -cert-err58-cpp,
-                    -cppcoreguidelines-avoid-const-or-ref-data-members.
+                    -cppcoreguidelines-avoid-const-or-ref-data-members,
                     -cppcoreguidelines-avoid-magic-numbers,
                     -cppcoreguidelines-avoid-non-const-global-variables,
                     -cppcoreguidelines-non-private-member-variables-in-classes,

diff --git a/lib/mls_vectors/src/mls_vectors.cpp b/lib/mls_vectors/src/mls_vectors.cpp
@@ -1236,18 +1236,27 @@ struct TreeTestCase
         // evaulating the "bugprone-unchecked-optional-access" lint.
         const auto& leaf = pair.first;
         auto& priv_state = pair.second;
+        if (leaf == from) {
+          priv_state =
+            PrivateState{ priv_state.sig_priv, sender_priv, { from } };
+          continue;
+        }
 
-        // Look up the path secret for the joiner
-        if (!joiner.empty()) {
-          auto index = joiner.front();
-          auto [overlap, shared_path_secret, ok] =
-            sender_priv.shared_path_secret(index);
-          silence_unused(overlap);
-          silence_unused(ok);
+        priv_state.priv.decap(from, pub_before, context, path, joiner);
+        priv_state.senders.push_back(from);
+      }
 
-          path_secret = shared_path_secret;
-        }
+      // Look up the path secret for the joiner
+      if (!joiner.empty()) {
+        auto index = joiner.front();
+        auto [overlap, shared_path_secret, ok] =
+          sender_priv.shared_path_secret(index);
+        silence_unused(overlap);
+        silence_unused(ok);
+
+        path_secret = shared_path_secret;
       }
+    }
 
     // Add a private entry for the joiner if we added someone
     if (!joiner.empty()) {