Commit a94e8258 authored by Ulrich Sibiller's avatar Ulrich Sibiller

os: Fix -Wshadow errors

commit 08d0481e299c28b64a0db9bb0782ba2b551028fd Author: Keith Packard <keithp@keithp.com> Date: Sun Dec 15 01:12:22 2013 -0800 os: Fix -Wshadow errors Rename variables to avoid shadowing globals Signed-off-by: 's avatarKeith Packard <keithp@keithp.com> Reviewed-by: 's avatarEric Anholt <eric@anholt.net>
parent 4b8194e1
...@@ -61,7 +61,7 @@ static Bool authFromXDMCP; ...@@ -61,7 +61,7 @@ static Bool authFromXDMCP;
static XdmAuthKeyRec privateKey; static XdmAuthKeyRec privateKey;
static char XdmAuthenticationName[] = "XDM-AUTHENTICATION-1"; static char XdmAuthenticationName[] = "XDM-AUTHENTICATION-1";
#define XdmAuthenticationNameLen (sizeof XdmAuthenticationName - 1) #define XdmAuthenticationNameLen (sizeof XdmAuthenticationName - 1)
static XdmAuthKeyRec rho; static XdmAuthKeyRec global_rho;
static Bool static Bool
XdmAuthenticationValidator (ARRAY8Ptr privateData, ARRAY8Ptr incomingData, XdmAuthenticationValidator (ARRAY8Ptr privateData, ARRAY8Ptr incomingData,
...@@ -76,7 +76,7 @@ XdmAuthenticationValidator (ARRAY8Ptr privateData, ARRAY8Ptr incomingData, ...@@ -76,7 +76,7 @@ XdmAuthenticationValidator (ARRAY8Ptr privateData, ARRAY8Ptr incomingData,
return FALSE; return FALSE;
incoming = (XdmAuthKeyPtr) incomingData->data; incoming = (XdmAuthKeyPtr) incomingData->data;
XdmcpDecrementKey (incoming); XdmcpDecrementKey (incoming);
return XdmcpCompareKeys (incoming, &rho); return XdmcpCompareKeys (incoming, &global_rho);
} }
return FALSE; return FALSE;
} }
...@@ -89,7 +89,7 @@ XdmAuthenticationGenerator (ARRAY8Ptr privateData, ARRAY8Ptr outgoingData, ...@@ -89,7 +89,7 @@ XdmAuthenticationGenerator (ARRAY8Ptr privateData, ARRAY8Ptr outgoingData,
outgoingData->data = 0; outgoingData->data = 0;
if (packet_type == REQUEST) { if (packet_type == REQUEST) {
if (XdmcpAllocARRAY8 (outgoingData, 8)) if (XdmcpAllocARRAY8 (outgoingData, 8))
XdmcpWrap ((unsigned char *)&rho, (unsigned char *)&privateKey, XdmcpWrap ((unsigned char *)&global_rho, (unsigned char *)&privateKey,
outgoingData->data, 8); outgoingData->data, 8);
} }
return TRUE; return TRUE;
...@@ -152,10 +152,10 @@ XdmAuthenticationInit (char *cookie, int cookie_len) ...@@ -152,10 +152,10 @@ XdmAuthenticationInit (char *cookie, int cookie_len)
cookie_len = 7; cookie_len = 7;
memmove (privateKey.data + 1, cookie, cookie_len); memmove (privateKey.data + 1, cookie, cookie_len);
} }
XdmcpGenerateKey (&rho); XdmcpGenerateKey (&global_rho);
XdmcpRegisterAuthentication (XdmAuthenticationName, XdmAuthenticationNameLen, XdmcpRegisterAuthentication (XdmAuthenticationName, XdmAuthenticationNameLen,
(char *)&rho, (char *)&global_rho,
sizeof (rho), sizeof (global_rho),
(ValidatorFunc)XdmAuthenticationValidator, (ValidatorFunc)XdmAuthenticationValidator,
(GeneratorFunc)XdmAuthenticationGenerator, (GeneratorFunc)XdmAuthenticationGenerator,
(AddAuthorFunc)XdmAuthenticationAddAuth); (AddAuthorFunc)XdmAuthenticationAddAuth);
...@@ -340,7 +340,7 @@ XdmAddCookie (unsigned short data_length, char *data, XID id) ...@@ -340,7 +340,7 @@ XdmAddCookie (unsigned short data_length, char *data, XID id)
{ {
/* R5 xdm sent bogus authorization data in the accept packet, /* R5 xdm sent bogus authorization data in the accept packet,
* but we can recover */ * but we can recover */
rho_bits = rho.data; rho_bits = global_rho.data;
key_bits = (unsigned char *) data; key_bits = (unsigned char *) data;
key_bits[0] = '\0'; key_bits[0] = '\0';
} }
...@@ -353,7 +353,7 @@ XdmAddCookie (unsigned short data_length, char *data, XID id) ...@@ -353,7 +353,7 @@ XdmAddCookie (unsigned short data_length, char *data, XID id)
break; break;
#ifdef XDMCP #ifdef XDMCP
case 8: /* auth from XDMCP is 8 bytes long */ case 8: /* auth from XDMCP is 8 bytes long */
rho_bits = rho.data; rho_bits = global_rho.data;
key_bits = (unsigned char *) data; key_bits = (unsigned char *) data;
break; break;
#endif #endif
...@@ -477,7 +477,7 @@ XdmRemoveCookie (unsigned short data_length, char *data) ...@@ -477,7 +477,7 @@ XdmRemoveCookie (unsigned short data_length, char *data)
break; break;
#ifdef XDMCP #ifdef XDMCP
case 8: case 8:
rho_bits = &rho; rho_bits = &global_rho;
key_bits = (XdmAuthKeyPtr) data; key_bits = (XdmAuthKeyPtr) data;
break; break;
#endif #endif
......
...@@ -679,15 +679,15 @@ XdmcpBlockHandler( ...@@ -679,15 +679,15 @@ XdmcpBlockHandler(
struct timeval **wt, struct timeval **wt,
void * pReadmask) void * pReadmask)
{ {
fd_set *LastSelectMask = (fd_set*)pReadmask; fd_set *last_select_mask = (fd_set*)pReadmask;
CARD32 millisToGo; CARD32 millisToGo;
if (state == XDM_OFF) if (state == XDM_OFF)
return; return;
FD_SET(xdmcpSocket, LastSelectMask); FD_SET(xdmcpSocket, last_select_mask);
#if defined(IPv6) && defined(AF_INET6) #if defined(IPv6) && defined(AF_INET6)
if (xdmcpSocket6 >= 0) if (xdmcpSocket6 >= 0)
FD_SET(xdmcpSocket6, LastSelectMask); FD_SET(xdmcpSocket6, last_select_mask);
#endif #endif
if (timeOutTime == 0) if (timeOutTime == 0)
return; return;
...@@ -710,7 +710,7 @@ XdmcpWakeupHandler( ...@@ -710,7 +710,7 @@ XdmcpWakeupHandler(
int i, int i,
void * pReadmask) void * pReadmask)
{ {
fd_set* LastSelectMask = (fd_set*)pReadmask; fd_set* last_select_mask = (fd_set*)pReadmask;
#ifdef NX_TRANS_SOCKET #ifdef NX_TRANS_SOCKET
...@@ -723,16 +723,16 @@ XdmcpWakeupHandler( ...@@ -723,16 +723,16 @@ XdmcpWakeupHandler(
return; return;
if (i > 0) if (i > 0)
{ {
if (FD_ISSET(xdmcpSocket, LastSelectMask)) if (FD_ISSET(xdmcpSocket, last_select_mask))
{ {
receive_packet(xdmcpSocket); receive_packet(xdmcpSocket);
FD_CLR(xdmcpSocket, LastSelectMask); FD_CLR(xdmcpSocket, last_select_mask);
} }
#if defined(IPv6) && defined(AF_INET6) #if defined(IPv6) && defined(AF_INET6)
if (xdmcpSocket6 >= 0 && FD_ISSET(xdmcpSocket6, LastSelectMask)) if (xdmcpSocket6 >= 0 && FD_ISSET(xdmcpSocket6, last_select_mask))
{ {
receive_packet(xdmcpSocket6); receive_packet(xdmcpSocket6);
FD_CLR(xdmcpSocket6, LastSelectMask); FD_CLR(xdmcpSocket6, last_select_mask);
} }
#endif #endif
} }
...@@ -757,12 +757,12 @@ static void ...@@ -757,12 +757,12 @@ static void
XdmcpSelectHost( XdmcpSelectHost(
struct sockaddr *host_sockaddr, struct sockaddr *host_sockaddr,
int host_len, int host_len,
ARRAY8Ptr AuthenticationName) ARRAY8Ptr auth_name)
{ {
state = XDM_START_CONNECTION; state = XDM_START_CONNECTION;
memmove(&req_sockaddr, host_sockaddr, host_len); memmove(&req_sockaddr, host_sockaddr, host_len);
req_socklen = host_len; req_socklen = host_len;
XdmcpSetAuthentication (AuthenticationName); XdmcpSetAuthentication (auth_name);
send_packet(); send_packet();
} }
...@@ -777,11 +777,11 @@ static void ...@@ -777,11 +777,11 @@ static void
XdmcpAddHost( XdmcpAddHost(
struct sockaddr *from, struct sockaddr *from,
int fromlen, int fromlen,
ARRAY8Ptr AuthenticationName, ARRAY8Ptr auth_name,
ARRAY8Ptr hostname, ARRAY8Ptr hostname,
ARRAY8Ptr status) ARRAY8Ptr status)
{ {
XdmcpSelectHost(from, fromlen, AuthenticationName); XdmcpSelectHost(from, fromlen, auth_name);
} }
/* /*
...@@ -1066,8 +1066,6 @@ send_query_msg(void) ...@@ -1066,8 +1066,6 @@ send_query_msg(void)
XdmcpWriteARRAYofARRAY8 (&buffer, &AuthenticationNames); XdmcpWriteARRAYofARRAY8 (&buffer, &AuthenticationNames);
if (broadcast) if (broadcast)
{ {
int i;
for (i = 0; i < NumBroadcastAddresses; i++) for (i = 0; i < NumBroadcastAddresses; i++)
XdmcpFlush (xdmcpSocket, &buffer, (XdmcpNetaddr) &BroadcastAddresses[i], XdmcpFlush (xdmcpSocket, &buffer, (XdmcpNetaddr) &BroadcastAddresses[i],
sizeof (struct sockaddr_in)); sizeof (struct sockaddr_in));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment