summaryrefslogtreecommitdiff
path: root/src/network
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2013-11-22 21:50:43 +0000
committerrubidium <rubidium@openttd.org>2013-11-22 21:50:43 +0000
commit2e54c8fdfa237f84e8d48b30bf4811ba7ed84327 (patch)
tree7ce2441341e514ccc4225d692f5c8d64b881e75c /src/network
parentbdd62a4b3e3df6282122a7c4353e6fef632f9c32 (diff)
downloadopenttd-2e54c8fdfa237f84e8d48b30bf4811ba7ed84327.tar.xz
(svn r26056) -Fix: a number of possibly uninitialised variables
Diffstat (limited to 'src/network')
-rw-r--r--src/network/core/tcp_admin.cpp4
-rw-r--r--src/network/core/tcp_game.cpp8
-rw-r--r--src/network/core/tcp_game.h2
-rw-r--r--src/network/network_content.cpp3
-rw-r--r--src/network/network_internal.h2
5 files changed, 11 insertions, 8 deletions
diff --git a/src/network/core/tcp_admin.cpp b/src/network/core/tcp_admin.cpp
index 0a3153319..636696e2c 100644
--- a/src/network/core/tcp_admin.cpp
+++ b/src/network/core/tcp_admin.cpp
@@ -29,9 +29,11 @@ assert_compile((int)CRR_END == (int)ADMIN_CRR_END);
* Create the admin handler for the given socket.
* @param s The socket to communicate over.
*/
-NetworkAdminSocketHandler::NetworkAdminSocketHandler(SOCKET s)
+NetworkAdminSocketHandler::NetworkAdminSocketHandler(SOCKET s) : status(ADMIN_STATUS_INACTIVE)
{
this->sock = s;
+ this->admin_name[0] = '\0';
+ this->admin_version[0] = '\0';
}
NetworkAdminSocketHandler::~NetworkAdminSocketHandler()
diff --git a/src/network/core/tcp_game.cpp b/src/network/core/tcp_game.cpp
index 711d6a2b7..54d222495 100644
--- a/src/network/core/tcp_game.cpp
+++ b/src/network/core/tcp_game.cpp
@@ -26,12 +26,10 @@
* Create a new socket for the game connection.
* @param s The socket to connect with.
*/
-NetworkGameSocketHandler::NetworkGameSocketHandler(SOCKET s)
+NetworkGameSocketHandler::NetworkGameSocketHandler(SOCKET s) : info(NULL),
+ last_frame(_frame_counter), last_frame_server(_frame_counter), last_packet(_realtime_tick)
{
- this->sock = s;
- this->last_frame = _frame_counter;
- this->last_frame_server = _frame_counter;
- this->last_packet = _realtime_tick;
+ this->sock = s;
}
/**
diff --git a/src/network/core/tcp_game.h b/src/network/core/tcp_game.h
index 761399639..5c6cb5c34 100644
--- a/src/network/core/tcp_game.h
+++ b/src/network/core/tcp_game.h
@@ -137,7 +137,7 @@ class CommandQueue {
public:
/** Initialise the command queue. */
- CommandQueue() : first(NULL), last(NULL) {}
+ CommandQueue() : first(NULL), last(NULL), count(0) {}
/** Clear the command queue. */
~CommandQueue() { this->Free(); }
void Append(CommandPacket *p);
diff --git a/src/network/network_content.cpp b/src/network/network_content.cpp
index 047269969..8677a8104 100644
--- a/src/network/network_content.cpp
+++ b/src/network/network_content.cpp
@@ -705,7 +705,8 @@ ClientNetworkContentSocketHandler::ClientNetworkContentSocketHandler() :
http_response_index(-2),
curFile(NULL),
curInfo(NULL),
- isConnecting(false)
+ isConnecting(false),
+ lastActivity(_realtime_tick)
{
}
diff --git a/src/network/network_internal.h b/src/network/network_internal.h
index 938682981..e46f20792 100644
--- a/src/network/network_internal.h
+++ b/src/network/network_internal.h
@@ -154,6 +154,8 @@ bool IsNetworkCompatibleVersion(const char *version);
* Everything we need to know about a command to be able to execute it.
*/
struct CommandPacket : CommandContainer {
+ /** Make sure the pointer is NULL. */
+ CommandPacket() : next(NULL) {}
CommandPacket *next; ///< the next command packet (if in queue)
CompanyByte company; ///< company that is executing the command
uint32 frame; ///< the frame in which this packet is executed