[PATCH 14/14] os: Fix -Wshadow warnings

Adam Jackson ajax at redhat.com
Fri Dec 13 15:05:52 PST 2013


Signed-off-by: Adam Jackson <ajax at redhat.com>
---
 os/xdmauth.c |  4 ++--
 os/xdmcp.c   | 28 +++++++++++++---------------
 2 files changed, 15 insertions(+), 17 deletions(-)

diff --git a/os/xdmauth.c b/os/xdmauth.c
index 28584d2..6fe259a 100644
--- a/os/xdmauth.c
+++ b/os/xdmauth.c
@@ -241,7 +241,7 @@ XdmClientAuthTimeout(long now)
 
 static XdmClientAuthPtr
 XdmAuthorizationValidate(unsigned char *plain, int length,
-                         XdmAuthKeyPtr rho, ClientPtr xclient,
+                         XdmAuthKeyPtr _rho, ClientPtr xclient,
                          const char **reason)
 {
     XdmClientAuthPtr client, existing;
@@ -257,7 +257,7 @@ XdmAuthorizationValidate(unsigned char *plain, int length,
     if (!client)
         return NULL;
     XdmClientAuthDecode(plain, client);
-    if (!XdmcpCompareKeys(&client->rho, rho)) {
+    if (!XdmcpCompareKeys(&client->rho, _rho)) {
         free(client);
         if (reason)
             *reason = "Invalid XDM-AUTHORIZATION-1 key (failed key comparison)";
diff --git a/os/xdmcp.c b/os/xdmcp.c
index 11f1133..58a6e7a 100644
--- a/os/xdmcp.c
+++ b/os/xdmcp.c
@@ -638,15 +638,15 @@ XdmcpCloseDisplay(int sock)
 XdmcpBlockHandler(pointer data, /* unused */
                   struct timeval **wt, pointer pReadmask)
 {
-    fd_set *LastSelectMask = (fd_set *) pReadmask;
+    fd_set *last_select_mask = (fd_set *) pReadmask;
     CARD32 millisToGo;
 
     if (state == XDM_OFF)
         return;
-    FD_SET(xdmcpSocket, LastSelectMask);
+    FD_SET(xdmcpSocket, last_select_mask);
 #if defined(IPv6) && defined(AF_INET6)
     if (xdmcpSocket6 >= 0)
-        FD_SET(xdmcpSocket6, LastSelectMask);
+        FD_SET(xdmcpSocket6, last_select_mask);
 #endif
     if (timeOutTime == 0)
         return;
@@ -666,23 +666,23 @@ XdmcpBlockHandler(pointer data, /* unused */
 XdmcpWakeupHandler(pointer data,        /* unused */
                    int i, pointer pReadmask)
 {
-    fd_set *LastSelectMask = (fd_set *) pReadmask;
+    fd_set *last_select_mask = (fd_set *) pReadmask;
     fd_set devicesReadable;
 
     if (state == XDM_OFF)
         return;
     if (i > 0) {
-        if (FD_ISSET(xdmcpSocket, LastSelectMask)) {
+        if (FD_ISSET(xdmcpSocket, last_select_mask)) {
             receive_packet(xdmcpSocket);
-            FD_CLR(xdmcpSocket, LastSelectMask);
+            FD_CLR(xdmcpSocket, last_select_mask);
         }
 #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);
-            FD_CLR(xdmcpSocket6, LastSelectMask);
+            FD_CLR(xdmcpSocket6, last_select_mask);
         }
 #endif
-        XFD_ANDSET(&devicesReadable, LastSelectMask, &EnabledDevices);
+        XFD_ANDSET(&devicesReadable, last_select_mask, &EnabledDevices);
         if (XFD_ANYSET(&devicesReadable)) {
             if (state == XDM_AWAIT_USER_INPUT)
                 restart();
@@ -709,12 +709,12 @@ XdmcpWakeupHandler(pointer data,        /* unused */
 
 static void
 XdmcpSelectHost(const struct sockaddr *host_sockaddr,
-                int host_len, ARRAY8Ptr AuthenticationName)
+                int host_len, ARRAY8Ptr authname)
 {
     state = XDM_START_CONNECTION;
     memmove(&req_sockaddr, host_sockaddr, host_len);
     req_socklen = host_len;
-    XdmcpSetAuthentication(AuthenticationName);
+    XdmcpSetAuthentication(authname);
     send_packet();
 }
 
@@ -727,9 +727,9 @@ XdmcpSelectHost(const struct sockaddr *host_sockaddr,
  /*ARGSUSED*/ static void
 XdmcpAddHost(const struct sockaddr *from,
              int fromlen,
-             ARRAY8Ptr AuthenticationName, ARRAY8Ptr hostname, ARRAY8Ptr status)
+             ARRAY8Ptr authname, ARRAY8Ptr hostname, ARRAY8Ptr status)
 {
-    XdmcpSelectHost(from, fromlen, AuthenticationName);
+    XdmcpSelectHost(from, fromlen, authname);
 }
 
 /*
@@ -1055,8 +1055,6 @@ send_query_msg(void)
     XdmcpWriteHeader(&buffer, &header);
     XdmcpWriteARRAYofARRAY8(&buffer, &AuthenticationNames);
     if (broadcast) {
-        int i;
-
         for (i = 0; i < NumBroadcastAddresses; i++)
             XdmcpFlush(xdmcpSocket, &buffer,
                        (XdmcpNetaddr) &BroadcastAddresses[i],
-- 
1.8.4.2



More information about the xorg-devel mailing list