diff options
author | Dario Lombardo <lomato@gmail.com> | 2016-03-04 10:56:00 +0100 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-03-07 05:10:15 +0000 |
commit | ef21d66edc4e3850f75368e99eb217cf84b475c6 (patch) | |
tree | dc966cf6ea50b35dd81ae3104ee4f54f5baf34b9 /extcap | |
parent | 7d543a0b873e0164c8c51a8cb7fc65502b0869ec (diff) |
sshdump: minor changes
Change-Id: I07bd2a95622187c6bacb3013998f93bfa07af87c
Reviewed-on: https://code.wireshark.org/review/14338
Petri-Dish: Dario Lombardo <lomato@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'extcap')
-rw-r--r-- | extcap/sshdump.c | 22 |
1 files changed, 10 insertions, 12 deletions
diff --git a/extcap/sshdump.c b/extcap/sshdump.c index 814328c1ab..d289229f88 100644 --- a/extcap/sshdump.c +++ b/extcap/sshdump.c @@ -208,8 +208,10 @@ static int ssh_open_remote_connection(const char* hostname, const unsigned int p sshs = create_ssh_connection(hostname, port, username, password, sshkey, sshkey_passphrase, &err_info); - if (!sshs) + if (!sshs) { + errmsg_print("Error creating connection: %s", err_info); goto cleanup; + } channel = run_ssh_command(sshs, capture_bin, iface, cfilter, count); if (!channel) @@ -368,19 +370,17 @@ int main(int argc, char **argv) char* remote_filter = NULL; unsigned long int count = 0; int ret = 0; - extcap_parameters * extcap_conf = g_new0(extcap_parameters, 1); - extcap_base_set_util_info(extcap_conf, SSHDUMP_VERSION_MAJOR, SSHDUMP_VERSION_MINOR, SSHDUMP_VERSION_RELEASE, NULL); - extcap_base_register_interface(extcap_conf, SSH_EXTCAP_INTERFACE, "SSH remote capture", 147, "Remote capture dependant DLT"); - - #ifdef _WIN32 WSADATA wsaData; attach_parent_console(); #endif /* _WIN32 */ + extcap_base_set_util_info(extcap_conf, SSHDUMP_VERSION_MAJOR, SSHDUMP_VERSION_MINOR, SSHDUMP_VERSION_RELEASE, NULL); + extcap_base_register_interface(extcap_conf, SSH_EXTCAP_INTERFACE, "SSH remote capture", 147, "Remote capture dependent DLT"); + opterr = 0; optind = 0; @@ -478,8 +478,7 @@ int main(int argc, char **argv) break; default: - if (!extcap_base_parse_options(extcap_conf, result - EXTCAP_OPT_LIST_INTERFACES, optarg)) - { + if (!extcap_base_parse_options(extcap_conf, result - EXTCAP_OPT_LIST_INTERFACES, optarg)) { errmsg_print("Invalid option: %s", argv[optind - 1]); return EXIT_FAILURE; } @@ -502,7 +501,7 @@ int main(int argc, char **argv) if (result != 0) { if (verbose) errmsg_print("ERROR: WSAStartup failed with error: %d", result); - return 1; + return EXIT_FAILURE; } #endif /* _WIN32 */ @@ -511,7 +510,7 @@ int main(int argc, char **argv) if (!remote_host) { errmsg_print("Missing parameter: --remote-host"); - return 1; + return EXIT_FAILURE; } filter = concat_filters(extcap_conf->capture_filter, remote_filter); ret = ssh_open_remote_connection(remote_host, remote_port, remote_username, @@ -520,7 +519,7 @@ int main(int argc, char **argv) g_free(filter); } else { verbose_print("You should not come here... maybe some parameter missing?\n"); - ret = 1; + ret = EXIT_FAILURE; } /* clean up stuff */ @@ -528,7 +527,6 @@ int main(int argc, char **argv) return ret; } - #ifdef _WIN32 int CALLBACK WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow) { |