Fix incorrect types

These types caused an incorrect signed/unsigned behaviour, so let's make
sure we use the appropriate type.

diff --git a/unix/vncserver/vncsession.c b/unix/vncserver/vncsession.c
index 875a57e9a23755cdc2f309278609dd795dea6fa1..c722922183ab1f91ee531b32d4d97454ad38bd78 100644 (file)
--- a/unix/vncserver/vncsession.c
+++ b/unix/vncserver/vncsession.c
@@ -349,7 +349,7 @@ static void
 redir_stdio(const char *homedir, const char *display)
 {
     int fd;
-    size_t hostlen;
+    long hostlen;
     char* hostname = NULL;
     char logfile[PATH_MAX];
 

diff --git a/unix/x0vncserver/x0vncserver.cxx b/unix/x0vncserver/x0vncserver.cxx
index a612f8acb34662afc9198c14c734918fcc3728a4..c89cd8537b1fc2942cb8f83345537d52a271c9eb 100644 (file)
--- a/unix/x0vncserver/x0vncserver.cxx
+++ b/unix/x0vncserver/x0vncserver.cxx
@@ -78,7 +78,7 @@ StringParameter interface("interface",
 
 static const char* defaultDesktopName()
 {
-  size_t host_max = sysconf(_SC_HOST_NAME_MAX);
+  long host_max = sysconf(_SC_HOST_NAME_MAX);
   if (host_max < 0)
     return "";
 
@@ -90,7 +90,7 @@ static const char* defaultDesktopName()
   if (pwent == NULL)
     return "";
 
-  size_t len = snprintf(NULL, 0, "%s@%s", pwent->pw_name, hostname.data());
+  int len = snprintf(NULL, 0, "%s@%s", pwent->pw_name, hostname.data());
   if (len < 0)
     return "";
 

diff --git a/vncviewer/win32.c b/vncviewer/win32.c
index e51acca333e3994b722a1a0c84fbc28068c5e734..77366f96b022bd8e893de48dfc1c2058819a7a27 100644 (file)
--- a/vncviewer/win32.c
+++ b/vncviewer/win32.c
@@ -142,7 +142,7 @@ void win32_disable_lowlevel_keyboard(HWND hwnd)
 }
 
 // Layout independent keys
-static const int vkey_map[][3] = {
+static const UINT vkey_map[][3] = {
   { VK_CANCEL,              NoSymbol,       XK_Break },
   { VK_BACK,                XK_BackSpace,   NoSymbol },
   { VK_TAB,                 XK_Tab,         NoSymbol },
@@ -233,7 +233,7 @@ static const int vkey_map[][3] = {
 // Layout dependent keys, but without useful symbols
 
 // Japanese
-static const int vkey_map_jp[][3] = {
+static const UINT vkey_map_jp[][3] = {
   { VK_KANA,                XK_Hiragana_Katakana, NoSymbol },
   { VK_KANJI,               XK_Kanji,       NoSymbol },
   { VK_OEM_ATTN,            XK_Eisu_toggle, NoSymbol },
@@ -248,12 +248,12 @@ static const int vkey_map_jp[][3] = {
 };
 
 // Korean
-static const int vkey_map_ko[][3] = {
+static const UINT vkey_map_ko[][3] = {
   { VK_HANGUL,              XK_Hangul,      NoSymbol },
   { VK_HANJA,               XK_Hangul_Hanja, NoSymbol },
 };
 
-static int lookup_vkey_map(UINT vkey, int extended, const int map[][3], size_t size)
+static int lookup_vkey_map(UINT vkey, int extended, const UINT map[][3], size_t size)
 {
   size_t i;
 

diff --git a/win/rfb_win32/RegConfig.cxx b/win/rfb_win32/RegConfig.cxx
index 715f809706845f959e91ebebf5987ea3ac1216c8..e697e66b1b612d20f41a89ff7456cbf25d6182cc 100644 (file)
--- a/win/rfb_win32/RegConfig.cxx
+++ b/win/rfb_win32/RegConfig.cxx
@@ -108,7 +108,7 @@ bool RegConfigThread::start(const HKEY rootKey, const TCHAR* keyname) {
 }
 
 void RegConfigThread::worker() {
-  DWORD result = 0;
+  BOOL result = 0;
   MSG msg;
   thread_id = GetCurrentThreadId();
   while ((result = eventMgr.getMessage(&msg, 0, 0, 0)) > 0) {}