diff options
author | imarom <imarom@cisco.com> | 2016-09-05 10:32:08 +0300 |
---|---|---|
committer | imarom <imarom@cisco.com> | 2016-09-07 14:02:57 +0300 |
commit | 0d4c9932591a46d5e2398dd1049001e107130f44 (patch) | |
tree | 71bbf0b7f287d721459132cefc91c104b989e4ef /src/rpc-server | |
parent | d508420652d613fdc87f6af746fd1372cd2ae7d2 (diff) |
dual mode : code review fixes
Diffstat (limited to 'src/rpc-server')
-rw-r--r-- | src/rpc-server/commands/trex_rpc_cmd_general.cpp | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/src/rpc-server/commands/trex_rpc_cmd_general.cpp b/src/rpc-server/commands/trex_rpc_cmd_general.cpp index 85f77a7f..342ec594 100644 --- a/src/rpc-server/commands/trex_rpc_cmd_general.cpp +++ b/src/rpc-server/commands/trex_rpc_cmd_general.cpp @@ -519,21 +519,12 @@ TrexRpcCmdPushRemote::_run(const Json::Value ¶ms, Json::Value &result) { TrexStatelessPort *port = get_stateless_obj()->get_port_by_id(port_id); + /* for dual mode - make sure slave_handler matches */ if (is_dual) { TrexStatelessPort *slave = get_stateless_obj()->get_port_by_id(port_id ^ 0x1); - if (!slave->get_owner().verify(slave_handler)) { generate_execute_err(result, "incorrect or missing slave port handler"); } - - std::stringstream ss; - CCapReaderBase *reader = CCapReaderFactory::CreateReader((char *)pcap_filename.c_str(), 0, ss); - if (!reader) { - generate_execute_err(result, ss.str()); - } - if (reader->get_type() != ERF) { - generate_execute_err(result, "dual mode is only supported on ERF format"); - } } |