diff options
author | Michael Mann <mmann78@netscape.net> | 2017-03-26 18:46:13 -0400 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-03-27 03:14:15 +0000 |
commit | 3b588dffcf4737f07ccf49baf6bc44a4108647c3 (patch) | |
tree | a30fab29f23656a7ec62e78cc38fb03f0d312dbe /dumpcap.c | |
parent | 520cfcf53b1c3e56ff524c89cf18fbf999a28f0c (diff) |
Check for return value of WSAStartup on all executables.
Windows vscodeanalysis complains even though the event is probably
very unlikely.
Change-Id: Iafe158eea5586908209d6bfe1e45540117558673
Reviewed-on: https://code.wireshark.org/review/20727
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'dumpcap.c')
-rw-r--r-- | dumpcap.c | 9 |
1 files changed, 8 insertions, 1 deletions
@@ -3779,6 +3779,7 @@ main(int argc, char *argv[]) gboolean arg_error = FALSE; #ifdef _WIN32 + int result; WSADATA wsaData; #else struct sigaction action, oldaction; @@ -3970,7 +3971,13 @@ main(int argc, char *argv[]) /*wpcap_packet_load();*/ /* Start windows sockets */ - WSAStartup( MAKEWORD( 1, 1 ), &wsaData ); + result = WSAStartup( MAKEWORD( 1, 1 ), &wsaData ); + if (result != 0) + { + g_log(LOG_DOMAIN_CAPTURE_CHILD, G_LOG_LEVEL_ERROR, + "ERROR: WSAStartup failed with error: %d", result); + exit_main(1); + } /* Set handler for Ctrl+C key */ SetConsoleCtrlHandler(capture_cleanup_handler, TRUE); |