rpc: Assert named arguments are unique in RPCHelpMan

maflcko · Dec 7, 2018 · e09a587 · e09a587
1 parent e2c473f
commit e09a587
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/src/rpc/util.cpp b/src/rpc/util.cpp
@@ -242,6 +242,16 @@ struct Sections {
     }
 };
 
+RPCHelpMan::RPCHelpMan(const std::string& name, const std::string& description, const std::vector<RPCArg>& args)
+    : m_name{name}, m_description{description}, m_args{args}
+{
+    std::set<std::string> named_args;
+    for (const auto& arg : m_args) {
+        // Should have unique named arguments
+        assert(named_args.insert(arg.m_name).second);
+    }
+}
+
 std::string RPCHelpMan::ToString() const
 {
     std::string ret;

diff --git a/src/rpc/util.h b/src/rpc/util.h
@@ -109,10 +109,7 @@ struct RPCArg {
 class RPCHelpMan
 {
 public:
-    RPCHelpMan(const std::string& name, const std::string& description, const std::vector<RPCArg>& args)
-        : m_name{name}, m_description{description}, m_args{args}
-    {
-    }
+    RPCHelpMan(const std::string& name, const std::string& description, const std::vector<RPCArg>& args);
 
     std::string ToString() const;