Fix various oversights/bugs in ZMQ RPC server code

- Add some RPC commands (and touch up a couple others)
- some bounds checking
- some better pointer management
- const correctness and error handling

-- Thanks @vtnerd for type help with serialization and CMake changes

1 files changed, 4 insertions, 5 deletions

diff --git a/src/rpc/zmq_server.h b/src/rpc/zmq_server.h
index 964ce2071..c278ff759 100644
--- a/src/rpc/zmq_server.h
+++ b/src/rpc/zmq_server.h
@@ -29,12 +29,11 @@
 #pragma once
 
 #include <boost/thread/thread.hpp>
-#include <boost/program_options/options_description.hpp>
-#include <boost/program_options/variables_map.hpp>
-#include "common/command_line.h"
-
 #include <zmq.hpp>
 #include <string>
+#include <memory>
+
+#include "common/command_line.h"
 
 #include "rpc_handler.h"
 
@@ -75,7 +74,7 @@ class ZmqServer
 
     boost::thread run_thread;
 
-    std::vector<zmq::socket_t*> sockets;
+    std::unique_ptr<zmq::socket_t> rep_socket;
 };