From 3b588dffcf4737f07ccf49baf6bc44a4108647c3 Mon Sep 17 00:00:00 2001 From: Michael Mann Date: Sun, 26 Mar 2017 18:46:13 -0400 Subject: 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 --- rawshark.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'rawshark.c') diff --git a/rawshark.c b/rawshark.c index e319acd01c..032ee93a7c 100644 --- a/rawshark.c +++ b/rawshark.c @@ -429,6 +429,7 @@ main(int argc, char *argv[]) gboolean arg_error = FALSE; #ifdef _WIN32 + int result; WSADATA wsaData; #else struct rlimit limit; @@ -824,7 +825,12 @@ main(int argc, char *argv[]) #ifdef _WIN32 /* Start windows sockets */ - WSAStartup( MAKEWORD( 1, 1 ), &wsaData ); + result = WSAStartup( MAKEWORD( 1, 1 ), &wsaData ); + if (result != 0) + { + ret = INIT_ERROR; + goto clean_exit; + } #endif /* _WIN32 */ /* At this point MATE will have registered its field array so we can -- cgit v1.2.1