upload tizen2.0 source
[framework/uifw/xorg/lib/xtrans.git] / Xtranslcl.c
old mode 100755 (executable)
new mode 100644 (file)
index ca04e7f..5beef7c
@@ -49,7 +49,7 @@ from The Open Group.
 
 /*
  *
- * The connection code/ideas in lib/X and server/os for SVR4/Intel 
+ * The connection code/ideas in lib/X and server/os for SVR4/Intel
  * environments was contributed by the following companies/groups:
  *
  *     MetroLink Inc
@@ -61,12 +61,12 @@ from The Open Group.
  *
  * The goal is to have common connection code among all SVR4/Intel vendors.
  *
- * ALL THE ABOVE COMPANIES DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS 
- * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, 
- * IN NO EVENT SHALL THESE COMPANIES * BE LIABLE FOR ANY SPECIAL, INDIRECT 
- * OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS 
- * OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE 
- * OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE 
+ * ALL THE ABOVE COMPANIES DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS
+ * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS,
+ * IN NO EVENT SHALL THESE COMPANIES * BE LIABLE FOR ANY SPECIAL, INDIRECT
+ * OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS
+ * OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
+ * OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE
  * OR PERFORMANCE OF THIS SOFTWARE.
  */
 
@@ -123,7 +123,7 @@ static int TRANS(LocalClose)(XtransConnInfo ciptr);
 /* Type Not Supported */
 
 static int
-TRANS(OpenFail)(XtransConnInfo ciptr, char *port)
+TRANS(OpenFail)(XtransConnInfo ciptr _X_UNUSED, char *port _X_UNUSED)
 
 {
     return -1;
@@ -132,7 +132,7 @@ TRANS(OpenFail)(XtransConnInfo ciptr, char *port)
 #ifdef TRANS_REOPEN
 
 static int
-TRANS(ReopenFail)(XtransConnInfo ciptr, int fd, char *port)
+TRANS(ReopenFail)(XtransConnInfo ciptr _X_UNUSED, int fd _X_UNUSED, char *port _X_UNUSED)
 
 {
     return 0;
@@ -152,21 +152,21 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr, char *sun_path, char *peer_sun_path)
     ciptr->family = AF_UNIX;
     ciptr->addrlen = sizeof (struct sockaddr_un);
 
-    if ((sunaddr = (struct sockaddr_un *) xalloc (ciptr->addrlen)) == NULL)
+    if ((sunaddr = malloc (ciptr->addrlen)) == NULL)
     {
-       PRMSG(1,"FillAddrInfo: failed to allocate memory for addr\n", 0, 0, 0);
+       prmsg(1,"FillAddrInfo: failed to allocate memory for addr\n");
        return 0;
     }
 
     sunaddr->sun_family = AF_UNIX;
 
     if (strlen(sun_path) > sizeof(sunaddr->sun_path) - 1) {
-       PRMSG(1, "FillAddrInfo: path too long\n", 0, 0, 0);
-       xfree((char *) sunaddr);
+       prmsg(1, "FillAddrInfo: path too long\n");
+       free((char *) sunaddr);
        return 0;
     }
     strcpy (sunaddr->sun_path, sun_path);
-#if defined(BSD44SOCKETS) 
+#if defined(BSD44SOCKETS)
     sunaddr->sun_len = strlen (sunaddr->sun_path);
 #endif
 
@@ -174,13 +174,11 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr, char *sun_path, char *peer_sun_path)
 
     ciptr->peeraddrlen = sizeof (struct sockaddr_un);
 
-    if ((p_sunaddr = (struct sockaddr_un *) xalloc (
-       ciptr->peeraddrlen)) == NULL)
+    if ((p_sunaddr = malloc (ciptr->peeraddrlen)) == NULL)
     {
-       PRMSG(1,
-          "FillAddrInfo: failed to allocate memory for peer addr\n",
-                                                                       0,0,0);
-       xfree ((char *) sunaddr);
+       prmsg(1,
+          "FillAddrInfo: failed to allocate memory for peer addr\n");
+       free (sunaddr);
        ciptr->addr = NULL;
 
        return 0;
@@ -189,12 +187,12 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr, char *sun_path, char *peer_sun_path)
     p_sunaddr->sun_family = AF_UNIX;
 
     if (strlen(peer_sun_path) > sizeof(p_sunaddr->sun_path) - 1) {
-       PRMSG(1, "FillAddrInfo: peer path too long\n", 0, 0, 0);
-       xfree((char *) p_sunaddr);
+       prmsg(1, "FillAddrInfo: peer path too long\n");
+       free((char *) p_sunaddr);
        return 0;
     }
     strcpy (p_sunaddr->sun_path, peer_sun_path);
-#if defined(BSD44SOCKETS) 
+#if defined(BSD44SOCKETS)
     p_sunaddr->sun_len = strlen (p_sunaddr->sun_path);
 #endif
 
@@ -208,7 +206,7 @@ TRANS(FillAddrInfo)(XtransConnInfo ciptr, char *sun_path, char *peer_sun_path)
 #ifdef LOCAL_TRANS_PTS
 /* PTS */
 
-#if defined(SYSV) && !defined(__SCO__) 
+#if defined(SYSV) && !defined(__SCO__)
 #define SIGNAL_T int
 #else
 #define SIGNAL_T void
@@ -222,7 +220,7 @@ extern char *ptsname(
     int
 );
 
-static void _dummy(int sig)
+static void _dummy(int sig _X_UNUSED)
 
 {
 }
@@ -315,20 +313,22 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
     pid_t              saved_pid;
 #endif
 
-    PRMSG(2,"PTSOpenClient(%s)\n", port, 0,0 );
+    prmsg(2,"PTSOpenClient(%s)\n", port);
 
 #if !defined(PTSNODENAME)
-    PRMSG(1,"PTSOpenClient: Protocol is not supported by a pts connection\n", 0,0,0);
+    prmsg(1,"PTSOpenClient: Protocol is not supported by a pts connection\n");
     return -1;
 #else
     if (port && *port ) {
        if( *port == '/' ) { /* A full pathname */
-               (void) sprintf(server_path, "%s", port);
-           } else {
-               (void) sprintf(server_path, "%s%s", PTSNODENAME, port);
-           }
+           snprintf(server_path, sizeof(server_path), "%s", port);
+       } else {
+           snprintf(server_path, sizeof(server_path), "%s%s",
+                    PTSNODENAME, port);
+       }
     } else {
-       (void) sprintf(server_path, "%s%d", PTSNODENAME, getpid());
+       snprintf(server_path, sizeof(server_path), "%s%d",
+                PTSNODENAME, getpid());
     }
 
 
@@ -337,7 +337,7 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
      */
 
     if ((server = open (server_path, O_RDWR)) < 0) {
-       PRMSG(1,"PTSOpenClient: failed to open %s\n", server_path, 0,0);
+       prmsg(1,"PTSOpenClient: failed to open %s\n", server_path);
        return -1;
     }
 
@@ -347,7 +347,7 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
      */
 
     if ((fd = open(DEV_PTMX, O_RDWR)) < 0) {
-       PRMSG(1,"PTSOpenClient: failed to open %s\n", DEV_PTMX, 0,0);
+       prmsg(1,"PTSOpenClient: failed to open %s\n", DEV_PTMX);
        close(server);
        return(-1);
     }
@@ -358,7 +358,7 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
     slave = ptsname(fd); /* get name */
 
     if( slave == NULL ) {
-       PRMSG(1,"PTSOpenClient: failed to get ptsname()\n", 0,0,0);
+       prmsg(1,"PTSOpenClient: failed to get ptsname()\n");
        close(fd);
        close(server);
        return -1;
@@ -391,14 +391,14 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
     if (WIFEXITED(exitval) && WEXITSTATUS(exitval) != 0) {
        close(fd);
        close(server);
-       PRMSG(1, "PTSOpenClient: cannot set the owner of %s\n",
-             slave, 0, 0);
+       prmsg(1, "PTSOpenClient: cannot set the owner of %s\n",
+             slave);
        return(-1);
     }
     if (chmod(slave, 0666) < 0) {
        close(fd);
        close(server);
-       PRMSG(1,"PTSOpenClient: Cannot chmod %s\n", slave, 0,0);
+       prmsg(1,"PTSOpenClient: Cannot chmod %s\n", slave);
        return(-1);
     }
 
@@ -425,8 +425,8 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
     (void) signal(SIGALRM, savef);
 
     if (ret != 1) {
-       PRMSG(1,
-       "PTSOpenClient: failed to get acknoledgement from server\n", 0, 0, 0);
+       prmsg(1,
+       "PTSOpenClient: failed to get acknoledgement from server\n");
        (void) close(fd);
        fd = -1;
     }
@@ -437,7 +437,7 @@ TRANS(PTSOpenClient)(XtransConnInfo ciptr, char *port)
 
     if (TRANS(FillAddrInfo) (ciptr, slave, server_path) == 0)
     {
-       PRMSG(1,"PTSOpenClient: failed to fill in addr info\n", 0, 0, 0);
+       prmsg(1,"PTSOpenClient: failed to fill in addr info\n");
        close(fd);
        return -1;
     }
@@ -462,10 +462,10 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
     int mode;
 #endif
 
-    PRMSG(2,"PTSOpenServer(%s)\n", port, 0,0 );
+    prmsg(2,"PTSOpenServer(%s)\n", port);
 
 #if !defined(PTSNODENAME)
-    PRMSG(1,"PTSOpenServer: Protocol is not supported by a pts connection\n", 0,0,0);
+    prmsg(1,"PTSOpenServer: Protocol is not supported by a pts connection\n");
     return -1;
 #else
     if (port && *port ) {
@@ -484,8 +484,8 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
     mode = 0777;
 #endif
     if (trans_mkdir(X_STREAMS_DIR, mode) == -1) {
-       PRMSG (1, "PTSOpenServer: mkdir(%s) failed, errno = %d\n",
-              X_STREAMS_DIR, errno, 0);
+       prmsg (1, "PTSOpenServer: mkdir(%s) failed, errno = %d\n",
+              X_STREAMS_DIR, errno);
        return(-1);
     }
 
@@ -496,8 +496,8 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
         * prevent clients from trying to connect to the in-use PTS (which
         * is often in use by something other than another server).
         */
-       PRMSG(1, "PTSOpenServer: A server is already running on port %s\n", port, 0,0 );
-       PRMSG(1, "PTSOpenServer: Remove %s if this is incorrect.\n", server_path, 0,0 );
+       prmsg(1, "PTSOpenServer: A server is already running on port %s\n", port);
+       prmsg(1, "PTSOpenServer: Remove %s if this is incorrect.\n", server_path);
        close(fd);
        return(-1);
     }
@@ -508,7 +508,7 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
     unlink(server_path);
 
     if( (fd=open(DEV_PTMX, O_RDWR)) < 0) {
-       PRMSG(1, "PTSOpenServer: Unable to open %s\n", DEV_PTMX, 0,0 );
+       prmsg(1, "PTSOpenServer: Unable to open %s\n", DEV_PTMX);
        return(-1);
     }
 
@@ -516,25 +516,25 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
     unlockpt(fd);
 
     if( (slave=ptsname(fd)) == NULL) {
-       PRMSG(1, "PTSOpenServer: Unable to get slave device name\n", 0,0,0 );
+       prmsg(1, "PTSOpenServer: Unable to get slave device name\n");
        close(fd);
        return(-1);
     }
 
     if( link(slave,server_path) < 0 ) {
-       PRMSG(1, "PTSOpenServer: Unable to link %s to %s\n", slave, server_path,0 );
+       prmsg(1, "PTSOpenServer: Unable to link %s to %s\n", slave, server_path);
        close(fd);
        return(-1);
     }
 
     if( chmod(server_path, 0666) < 0 ) {
-       PRMSG(1, "PTSOpenServer: Unable to chmod %s to 0666\n", server_path,0,0 );
+       prmsg(1, "PTSOpenServer: Unable to chmod %s to 0666\n", server_path);
        close(fd);
        return(-1);
     }
 
     if( (server=open(server_path, O_RDWR)) < 0 ) {
-       PRMSG(1, "PTSOpenServer: Unable to open server device %s\n", server_path,0,0 );
+       prmsg(1, "PTSOpenServer: Unable to open server device %s\n", server_path);
        close(fd);
        return(-1);
     }
@@ -547,7 +547,7 @@ TRANS(PTSOpenServer)(XtransConnInfo ciptr, char *port)
 
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"PTSOpenServer: failed to fill in addr info\n", 0, 0, 0);
+       prmsg(1,"PTSOpenServer: failed to fill in addr info\n");
        close(fd);
        return -1;
     }
@@ -567,17 +567,17 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
     char               buf[256];
     struct sockaddr_un *sunaddr;
 
-    PRMSG(2,"PTSAccept(%x->%d)\n",ciptr,ciptr->fd,0);
+    prmsg(2,"PTSAccept(%x->%d)\n",ciptr,ciptr->fd);
 
     if( (in=read(ciptr->fd,&length,1)) <= 0 ){
        if( !in ) {
-               PRMSG(2,
-               "PTSAccept: Incoming connection closed\n",0,0,0);
+               prmsg(2,
+               "PTSAccept: Incoming connection closed\n");
                }
        else {
-               PRMSG(1,
+               prmsg(1,
        "PTSAccept: Error reading incoming connection. errno=%d \n",
-                                                               errno,0,0);
+                                                               errno);
                }
        *status = TRANS_ACCEPT_MISC_ERROR;
        return -1;
@@ -585,13 +585,13 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
 
     if( (in=read(ciptr->fd,buf,length)) <= 0 ){
        if( !in ) {
-               PRMSG(2,
-               "PTSAccept: Incoming connection closed\n",0,0,0);
+               prmsg(2,
+               "PTSAccept: Incoming connection closed\n");
                }
        else {
-               PRMSG(1,
+               prmsg(1,
 "PTSAccept: Error reading device name for new connection. errno=%d \n",
-                                                               errno,0,0);
+                                                               errno);
                }
        *status = TRANS_ACCEPT_MISC_ERROR;
        return -1;
@@ -600,7 +600,7 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
     buf[length] = '\0';
 
     if( (newfd=open(buf,O_RDWR)) < 0 ) {
-       PRMSG(1, "PTSAccept: Failed to open %s\n",buf,0,0);
+       prmsg(1, "PTSAccept: Failed to open %s\n",buf);
        *status = TRANS_ACCEPT_MISC_ERROR;
        return -1;
     }
@@ -612,9 +612,8 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
      */
 
     newciptr->addrlen=ciptr->addrlen;
-    if( (newciptr->addr=(char *)xalloc(newciptr->addrlen)) == NULL ) {
-       PRMSG(1,"PTSAccept: failed to allocate memory for peer addr\n",
-                                                                       0,0,0);
+    if( (newciptr->addr = malloc(newciptr->addrlen)) == NULL ) {
+       prmsg(1,"PTSAccept: failed to allocate memory for peer addr\n");
        close(newfd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -623,10 +622,9 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
     memcpy(newciptr->addr,ciptr->addr,newciptr->addrlen);
 
     newciptr->peeraddrlen=sizeof(struct sockaddr_un);
-    if( (sunaddr=(struct sockaddr_un *)xalloc(newciptr->peeraddrlen)) == NULL ) {
-       PRMSG(1,"PTSAccept: failed to allocate memory for peer addr\n",
-                                                                       0,0,0);
-       xfree(newciptr->addr);
+    if( (sunaddr = malloc(newciptr->peeraddrlen)) == NULL ) {
+       prmsg(1,"PTSAccept: failed to allocate memory for peer addr\n");
+       free(newciptr->addr);
        close(newfd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -634,7 +632,7 @@ TRANS(PTSAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
 
     sunaddr->sun_family=AF_UNIX;
     strcpy(sunaddr->sun_path,buf);
-#if defined(BSD44SOCKETS) 
+#if defined(BSD44SOCKETS)
     sunaddr->sun_len=strlen(sunaddr->sun_path);
 #endif
 
@@ -663,15 +661,15 @@ TRANS(NAMEDOpenClient)(XtransConnInfo ciptr, char *port)
     int                        fd;
     char               server_path[64];
     struct stat                filestat;
-# ifndef sun    
+# ifndef sun
     extern int         isastream(int);
 # endif
 #endif
 
-    PRMSG(2,"NAMEDOpenClient(%s)\n", port, 0,0 );
+    prmsg(2,"NAMEDOpenClient(%s)\n", port);
 
 #if !defined(NAMEDNODENAME)
-    PRMSG(1,"NAMEDOpenClient: Protocol is not supported by a NAMED connection\n", 0,0,0);
+    prmsg(1,"NAMEDOpenClient: Protocol is not supported by a NAMED connection\n");
     return -1;
 #else
     if ( port && *port ) {
@@ -685,26 +683,26 @@ TRANS(NAMEDOpenClient)(XtransConnInfo ciptr, char *port)
     }
 
     if ((fd = open(server_path, O_RDWR)) < 0) {
-       PRMSG(1,"NAMEDOpenClient: Cannot open %s for NAMED connection\n", server_path, 0,0 );
+       prmsg(1,"NAMEDOpenClient: Cannot open %s for NAMED connection\n", server_path);
        return -1;
     }
 
     if (fstat(fd, &filestat) < 0 ) {
-       PRMSG(1,"NAMEDOpenClient: Cannot stat %s for NAMED connection\n", server_path, 0,0 );
+       prmsg(1,"NAMEDOpenClient: Cannot stat %s for NAMED connection\n", server_path);
        (void) close(fd);
        return -1;
     }
 
     if ((filestat.st_mode & S_IFMT) != S_IFIFO) {
-       PRMSG(1,"NAMEDOpenClient: Device %s is not a FIFO\n", server_path, 0,0 );
+       prmsg(1,"NAMEDOpenClient: Device %s is not a FIFO\n", server_path);
        /* Is this really a failure? */
-       (void) close(fd);       
+       (void) close(fd);
        return -1;
     }
 
 
     if (isastream(fd) <= 0) {
-       PRMSG(1,"NAMEDOpenClient: %s is not a streams device\n", server_path, 0,0 );
+       prmsg(1,"NAMEDOpenClient: %s is not a streams device\n", server_path);
        (void) close(fd);
        return -1;
     }
@@ -715,8 +713,7 @@ TRANS(NAMEDOpenClient)(XtransConnInfo ciptr, char *port)
 
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"NAMEDOpenClient: failed to fill in addr info\n",
-                                                               0,0,0);
+       prmsg(1,"NAMEDOpenClient: failed to fill in addr info\n");
        close(fd);
        return -1;
     }
@@ -736,58 +733,54 @@ TRANS(NAMEDOpenClient)(XtransConnInfo ciptr, char *port)
 static int
 TRANS(NAMEDOpenPipe)(const char *server_path)
 {
-    PRMSG(2,"NAMEDOpenPipe(%s)\n", server_path, 0,0 );
-
     int                        fd, pipefd[2];
     struct stat                sbuf;
     int                        mode;
-    
-#if defined(sun) && defined(X11_t)
-    mode = 0775;       /* Solaris requires uid or gid 0 to create X11 pipes */
-#else    
+
+    prmsg(2,"NAMEDOpenPipe(%s)\n", server_path);
+
 #ifdef HAS_STICKY_DIR_BIT
     mode = 01777;
 #else
     mode = 0777;
 #endif
-#endif
     if (trans_mkdir(X_STREAMS_DIR, mode) == -1) {
-       PRMSG (1, "NAMEDOpenPipe: mkdir(%s) failed, errno = %d\n",
-              X_STREAMS_DIR, errno, 0);
+       prmsg (1, "NAMEDOpenPipe: mkdir(%s) failed, errno = %d\n",
+              X_STREAMS_DIR, errno);
        return(-1);
     }
 
     if(stat(server_path, &sbuf) != 0) {
        if (errno == ENOENT) {
            if ((fd = creat(server_path, (mode_t)0666)) == -1) {
-               PRMSG(1, "NAMEDOpenPipe: Can't open %s\n", server_path, 0,0 );
+               prmsg(1, "NAMEDOpenPipe: Can't open %s\n", server_path);
                return(-1);
            }
            close(fd);
            if (chmod(server_path, (mode_t)0666) < 0) {
-               PRMSG(1, "NAMEDOpenPipe: Can't open %s\n", server_path, 0,0 );
+               prmsg(1, "NAMEDOpenPipe: Can't open %s\n", server_path);
                return(-1);
            }
        } else {
-           PRMSG(1, "NAMEDOpenPipe: stat on %s failed\n", server_path, 0,0 );
+           prmsg(1, "NAMEDOpenPipe: stat on %s failed\n", server_path);
            return(-1);
        }
     }
 
     if( pipe(pipefd) != 0) {
-       PRMSG(1, "NAMEDOpenPipe: pipe() failed, errno=%d\n",errno, 0,0 );
+       prmsg(1, "NAMEDOpenPipe: pipe() failed, errno=%d\n",errno);
        return(-1);
     }
 
     if( ioctl(pipefd[0], I_PUSH, "connld") != 0) {
-       PRMSG(1, "NAMEDOpenPipe: ioctl(I_PUSH,\"connld\") failed, errno=%d\n",errno, 0,0 );
+       prmsg(1, "NAMEDOpenPipe: ioctl(I_PUSH,\"connld\") failed, errno=%d\n",errno);
        close(pipefd[0]);
        close(pipefd[1]);
        return(-1);
     }
 
     if( fattach(pipefd[0], server_path) != 0) {
-       PRMSG(1, "NAMEDOpenPipe: fattach(%s) failed, errno=%d\n", server_path,errno, 0 );
+       prmsg(1, "NAMEDOpenPipe: fattach(%s) failed, errno=%d\n", server_path,errno);
        close(pipefd[0]);
        close(pipefd[1]);
        return(-1);
@@ -805,10 +798,10 @@ TRANS(NAMEDOpenServer)(XtransConnInfo ciptr, char *port)
     char               server_path[64];
 #endif
 
-    PRMSG(2,"NAMEDOpenServer(%s)\n", port, 0,0 );
+    prmsg(2,"NAMEDOpenServer(%s)\n", port);
 
 #if !defined(NAMEDNODENAME)
-    PRMSG(1,"NAMEDOpenServer: Protocol is not supported by a NAMED connection\n", 0,0,0);
+    prmsg(1,"NAMEDOpenServer: Protocol is not supported by a NAMED connection\n");
     return -1;
 #else
     if ( port && *port ) {
@@ -827,14 +820,14 @@ TRANS(NAMEDOpenServer)(XtransConnInfo ciptr, char *port)
     if (fd < 0) {
        return -1;
     }
-    
+
     /*
      * Everything looks good: fill in the XtransConnInfo structure.
      */
 
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"NAMEDOpenServer: failed to fill in addr info\n", 0,0,0);
+       prmsg(1,"NAMEDOpenServer: failed to fill in addr info\n");
        TRANS(LocalClose)(ciptr);
        return -1;
     }
@@ -848,12 +841,11 @@ static int
 TRANS(NAMEDResetListener) (XtransConnInfo ciptr)
 
 {
-  int status = TRANS_RESET_NOOP;
   struct sockaddr_un      *sockname=(struct sockaddr_un *) ciptr->addr;
   struct stat     statb;
-  
-  PRMSG(2,"NAMEDResetListener(%p, %d)\n", ciptr, ciptr->fd, 0 );
-  
+
+  prmsg(2,"NAMEDResetListener(%p, %d)\n", ciptr, ciptr->fd);
+
   if (ciptr->fd != -1) {
     /*
      * see if the pipe has disappeared
@@ -861,7 +853,7 @@ TRANS(NAMEDResetListener) (XtransConnInfo ciptr)
 
     if (stat (sockname->sun_path, &statb) == -1 ||
        (statb.st_mode & S_IFMT) != S_IFIFO) {
-      PRMSG(3, "Pipe %s trashed, recreating\n", sockname->sun_path, 0, 0);
+      prmsg(3, "Pipe %s trashed, recreating\n", sockname->sun_path);
       TRANS(LocalClose)(ciptr);
       ciptr->fd = TRANS(NAMEDOpenPipe)(sockname->sun_path);
       if (ciptr->fd >= 0)
@@ -879,10 +871,10 @@ TRANS(NAMEDAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
 {
     struct strrecvfd str;
 
-    PRMSG(2,"NAMEDAccept(%x->%d)\n", ciptr, ciptr->fd, 0 );
+    prmsg(2,"NAMEDAccept(%p->%d)\n", ciptr, ciptr->fd);
 
     if( ioctl(ciptr->fd, I_RECVFD, &str ) < 0 ) {
-       PRMSG(1, "NAMEDAccept: ioctl(I_RECVFD) failed, errno=%d\n", errno, 0,0 );
+       prmsg(1, "NAMEDAccept: ioctl(I_RECVFD) failed, errno=%d\n", errno);
        *status = TRANS_ACCEPT_MISC_ERROR;
        return(-1);
     }
@@ -892,10 +884,9 @@ TRANS(NAMEDAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
      */
     newciptr->family=ciptr->family;
     newciptr->addrlen=ciptr->addrlen;
-    if( (newciptr->addr=(char *)xalloc(newciptr->addrlen)) == NULL ) {
-       PRMSG(1,
-             "NAMEDAccept: failed to allocate memory for pipe addr\n",
-                                                                       0,0,0);
+    if( (newciptr->addr = malloc(newciptr->addrlen)) == NULL ) {
+       prmsg(1,
+             "NAMEDAccept: failed to allocate memory for pipe addr\n");
        close(str.fd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -904,11 +895,10 @@ TRANS(NAMEDAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
     memcpy(newciptr->addr,ciptr->addr,newciptr->addrlen);
 
     newciptr->peeraddrlen=newciptr->addrlen;
-    if( (newciptr->peeraddr=(char *)xalloc(newciptr->peeraddrlen)) == NULL ) {
-       PRMSG(1,
-       "NAMEDAccept: failed to allocate memory for peer addr\n",
-                                                                       0,0,0);
-       xfree(newciptr->addr);
+    if( (newciptr->peeraddr = malloc(newciptr->peeraddrlen)) == NULL ) {
+       prmsg(1,
+       "NAMEDAccept: failed to allocate memory for peer addr\n");
+       free(newciptr->addr);
        close(str.fd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -1019,23 +1009,23 @@ TRANS(SCOOpenClient)(XtransConnInfo ciptr, char *port)
     extern int getmsg(), putmsg();
 #endif
 
-    PRMSG(2,"SCOOpenClient(%s)\n", port, 0,0 );
+    prmsg(2,"SCOOpenClient(%s)\n", port);
     if (!port || !port[0])
        port = "0";
 
 #if !defined(SCORNODENAME)
-    PRMSG(2,"SCOOpenClient: Protocol is not supported by a SCO connection\n", 0,0,0);
+    prmsg(2,"SCOOpenClient: Protocol is not supported by a SCO connection\n");
     return -1;
 #else
     (void) sprintf(server_path, SCORNODENAME, port);
 
     if ((server = open(server_path, O_RDWR)) < 0) {
-       PRMSG(1,"SCOOpenClient: failed to open %s\n", server_path, 0,0 );
+       prmsg(1,"SCOOpenClient: failed to open %s\n", server_path);
        return -1;
     }
 
     if ((fd = open(DEV_SPX, O_RDWR)) < 0) {
-       PRMSG(1,"SCOOpenClient: failed to open %s\n", DEV_SPX, 0,0 );
+       prmsg(1,"SCOOpenClient: failed to open %s\n", DEV_SPX);
        close(server);
        return -1;
     }
@@ -1055,7 +1045,7 @@ TRANS(SCOOpenClient)(XtransConnInfo ciptr, char *port)
     (void) signal(SIGALRM, savef);
 
     if (ret < 0) {
-       PRMSG(1,"SCOOpenClient: error from getmsg\n", 0,0,0 );
+       prmsg(1,"SCOOpenClient: error from getmsg\n");
        close(fd);
        close(server);
        return -1;
@@ -1079,7 +1069,7 @@ TRANS(SCOOpenClient)(XtransConnInfo ciptr, char *port)
 #endif
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"SCOOpenClient: failed to fill addr info\n", 0, 0, 0);
+       prmsg(1,"SCOOpenClient: failed to fill addr info\n");
        close(fd);
        return -1;
     }
@@ -1105,12 +1095,12 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
     int                        fds = -1;
 #endif
 
-    PRMSG(2,"SCOOpenServer(%s)\n", port, 0,0 );
+    prmsg(2,"SCOOpenServer(%s)\n", port);
     if (!port || !port[0])
        port = "0";
 
 #if !defined(SCORNODENAME)
-    PRMSG(1,"SCOOpenServer: Protocol is not supported by a SCO connection\n", 0,0,0);
+    prmsg(1,"SCOOpenServer: Protocol is not supported by a SCO connection\n");
     return -1;
 #else
     (void) sprintf(serverR_path, SCORNODENAME, port);
@@ -1122,7 +1112,7 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
 
     if ((fds = open(DEV_SPX, O_RDWR)) < 0 ||
        (fdr = open(DEV_SPX, O_RDWR)) < 0 ) {
-       PRMSG(1,"SCOOpenServer: failed to open %s\n", DEV_SPX, 0,0 );
+       prmsg(1,"SCOOpenServer: failed to open %s\n", DEV_SPX);
        if (fds >= 0)
                close(fds);
        if (fdr >= 0)
@@ -1131,14 +1121,14 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
     }
 
     if (named_spipe (fds, serverS_path) == -1) {
-       PRMSG(1,"SCOOpenServer: failed to create %s\n", serverS_path, 0, 0);
+       prmsg(1,"SCOOpenServer: failed to create %s\n", serverS_path);
        close (fdr);
        close (fds);
        return -1;
     }
 
     if (named_spipe (fdr, serverR_path) == -1) {
-       PRMSG(1,"SCOOpenServer: failed to create %s\n", serverR_path, 0, 0);
+       prmsg(1,"SCOOpenServer: failed to create %s\n", serverR_path);
        close (fdr);
        close (fds);
        return -1;
@@ -1147,7 +1137,7 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
 
     fds = open (serverS_path, O_RDWR | O_NDELAY);
     if (fds < 0) {
-       PRMSG(1,"SCOOpenServer: failed to open %s\n", serverS_path, 0, 0);
+       prmsg(1,"SCOOpenServer: failed to open %s\n", serverS_path);
        return -1;
     }
 
@@ -1160,22 +1150,22 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
     mylock.l_start     = 0;
     mylock.l_len       = 0;
     if (fcntl (fds, F_SETLK, &mylock) < 0) {
-       PRMSG(1,"SCOOpenServer: failed to lock %s\n", serverS_path, 0, 0);
+       prmsg(1,"SCOOpenServer: failed to lock %s\n", serverS_path);
        close (fds);
        return -1;
     }
 
     fdr = open (serverR_path, O_RDWR | O_NDELAY);
     if (fdr < 0) {
-       PRMSG(1,"SCOOpenServer: failed to open %s\n", serverR_path, 0, 0);
+       prmsg(1,"SCOOpenServer: failed to open %s\n", serverR_path);
        close (fds);
        return -1;
     }
 #endif /* X11_t */
 
     if (connect_spipe(fds, fdr)) {
-       PRMSG(1,"SCOOpenServer: ioctl(I_FDINSERT) failed on %s\n",
-             serverS_path, 0, 0);
+       prmsg(1,"SCOOpenServer: ioctl(I_FDINSERT) failed on %s\n",
+             serverS_path);
        close (fdr);
        close (fds);
        return -1;
@@ -1189,7 +1179,7 @@ TRANS(SCOOpenServer)(XtransConnInfo ciptr, char *port)
     ciptr->flags |= TRANS_NOUNLINK;
 #endif
     if (TRANS(FillAddrInfo) (ciptr, serverS_path, serverR_path) == 0) {
-       PRMSG(1,"SCOOpenServer: failed to fill in addr info\n", 0,0,0);
+       prmsg(1,"SCOOpenServer: failed to fill in addr info\n");
        close(fds);
        close(fdr);
        return -1;
@@ -1206,22 +1196,22 @@ TRANS(SCOAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
     char               c;
     int                        fd;
 
-    PRMSG(2,"SCOAccept(%d)\n", ciptr->fd, 0,0 );
+    prmsg(2,"SCOAccept(%d)\n", ciptr->fd);
 
     if (read(ciptr->fd, &c, 1) < 0) {
-       PRMSG(1,"SCOAccept: can't read from client\n",0,0,0);
+       prmsg(1,"SCOAccept: can't read from client\n");
        *status = TRANS_ACCEPT_MISC_ERROR;
        return(-1);
     }
 
     if( (fd = open(DEV_SPX, O_RDWR)) < 0 ) {
-       PRMSG(1,"SCOAccept: can't open \"%s\"\n",DEV_SPX, 0,0 );
+       prmsg(1,"SCOAccept: can't open \"%s\"\n",DEV_SPX);
        *status = TRANS_ACCEPT_MISC_ERROR;
        return(-1);
     }
 
     if (connect_spipe (ciptr->fd, fd) < 0) {
-       PRMSG(1,"SCOAccept: ioctl(I_FDINSERT) failed\n", 0, 0, 0);
+       prmsg(1,"SCOAccept: ioctl(I_FDINSERT) failed\n");
        close (fd);
        *status = TRANS_ACCEPT_MISC_ERROR;
        return -1;
@@ -1232,10 +1222,9 @@ TRANS(SCOAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
      */
 
     newciptr->addrlen=ciptr->addrlen;
-    if( (newciptr->addr=(char *)xalloc(newciptr->addrlen)) == NULL ) {
-       PRMSG(1,
-             "SCOAccept: failed to allocate memory for peer addr\n",
-             0,0,0);
+    if( (newciptr->addr = malloc(newciptr->addrlen)) == NULL ) {
+       prmsg(1,
+             "SCOAccept: failed to allocate memory for peer addr\n");
        close(fd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -1247,11 +1236,10 @@ TRANS(SCOAccept)(XtransConnInfo ciptr, XtransConnInfo newciptr, int *status)
 #endif
 
     newciptr->peeraddrlen=newciptr->addrlen;
-    if( (newciptr->peeraddr=(char *)xalloc(newciptr->peeraddrlen)) == NULL ) {
-       PRMSG(1,
-             "SCOAccept: failed to allocate memory for peer addr\n",
-             0,0,0);
-       xfree(newciptr->addr);
+    if( (newciptr->peeraddr = malloc(newciptr->peeraddrlen)) == NULL ) {
+       prmsg(1,
+             "SCOAccept: failed to allocate memory for peer addr\n");
+       free(newciptr->addr);
        close(fd);
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return -1;
@@ -1280,26 +1268,27 @@ TRANS(PTSReopenServer)(XtransConnInfo ciptr, int fd, char *port)
     char server_path[64];
 #endif
 
-    PRMSG(2,"PTSReopenServer(%d,%s)\n", fd, port, 0 );
+    prmsg(2,"PTSReopenServer(%d,%s)\n", fd, port);
 
 #if !defined(PTSNODENAME)
-    PRMSG(1,"PTSReopenServer: Protocol is not supported by a pts connection\n", 0,0,0);
+    prmsg(1,"PTSReopenServer: Protocol is not supported by a pts connection\n");
     return 0;
 #else
     if (port && *port ) {
        if( *port == '/' ) { /* A full pathname */
-               (void) sprintf(server_path, "%s", port);
-           } else {
-               (void) sprintf(server_path, "%s%s", PTSNODENAME, port);
-           }
+           snprintf(server_path, sizeof(server_path), "%s", port);
+       } else {
+           snprintf(server_path, sizeof(server_path), "%s%s",
+                    PTSNODENAME, port);
+       }
     } else {
-       (void) sprintf(server_path, "%s%ld", PTSNODENAME, (long)getpid());
+       snprintf(server_path, sizeof(server_path), "%s%ld",
+               PTSNODENAME, (long)getpid());
     }
 
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"PTSReopenServer: failed to fill in addr info\n",
-                                                               0,0,0);
+       prmsg(1,"PTSReopenServer: failed to fill in addr info\n");
        return 0;
     }
 
@@ -1313,33 +1302,34 @@ TRANS(PTSReopenServer)(XtransConnInfo ciptr, int fd, char *port)
 #ifdef LOCAL_TRANS_NAMED
 
 static int
-TRANS(NAMEDReopenServer)(XtransConnInfo ciptr, int fd, char *port)
+TRANS(NAMEDReopenServer)(XtransConnInfo ciptr, int fd _X_UNUSED, char *port)
 
 {
 #ifdef NAMEDNODENAME
     char server_path[64];
 #endif
 
-    PRMSG(2,"NAMEDReopenServer(%s)\n", port, 0,0 );
+    prmsg(2,"NAMEDReopenServer(%s)\n", port);
 
 #if !defined(NAMEDNODENAME)
-    PRMSG(1,"NAMEDReopenServer: Protocol is not supported by a NAMED connection\n", 0,0,0);
+    prmsg(1,"NAMEDReopenServer: Protocol is not supported by a NAMED connection\n");
     return 0;
 #else
     if ( port && *port ) {
        if( *port == '/' ) { /* A full pathname */
-           (void) sprintf(server_path, "%s", port);
+           snprintf(server_path, sizeof(server_path),"%s", port);
        } else {
-           (void) sprintf(server_path, "%s%s", NAMEDNODENAME, port);
+           snprintf(server_path, sizeof(server_path), "%s%s",
+                    NAMEDNODENAME, port);
        }
     } else {
-       (void) sprintf(server_path, "%s%ld", NAMEDNODENAME, (long)getpid());
+       snprintf(server_path, sizeof(server_path), "%s%ld",
+               NAMEDNODENAME, (long)getpid());
     }
 
     if (TRANS(FillAddrInfo) (ciptr, server_path, server_path) == 0)
     {
-       PRMSG(1,"NAMEDReopenServer: failed to fill in addr info\n",
-                                                               0,0,0);
+       prmsg(1,"NAMEDReopenServer: failed to fill in addr info\n");
        return 0;
     }
 
@@ -1360,12 +1350,12 @@ TRANS(SCOReopenServer)(XtransConnInfo ciptr, int fd, char *port)
     char serverR_path[64], serverS_path[64];
 #endif
 
-    PRMSG(2,"SCOReopenServer(%s)\n", port, 0,0 );
+    prmsg(2,"SCOReopenServer(%s)\n", port);
     if (!port || !port[0])
       port = "0";
 
 #if !defined(SCORNODENAME)
-    PRMSG(2,"SCOReopenServer: Protocol is not supported by a SCO connection\n", 0,0,0);
+    prmsg(2,"SCOReopenServer: Protocol is not supported by a SCO connection\n");
     return 0;
 #else
     (void) sprintf(serverR_path, SCORNODENAME, port);
@@ -1376,7 +1366,7 @@ TRANS(SCOReopenServer)(XtransConnInfo ciptr, int fd, char *port)
 #endif
     if (TRANS(FillAddrInfo) (ciptr, serverS_path, serverR_path) == 0)
     {
-       PRMSG(1, "SCOReopenServer: failed to fill in addr info\n", 0,0,0);
+       prmsg(1, "SCOReopenServer: failed to fill in addr info\n");
        return 0;
     }
 
@@ -1397,7 +1387,7 @@ TRANS(SCOReopenServer)(XtransConnInfo ciptr, int fd, char *port)
  */
 
 typedef struct _LOCALtrans2dev {
-    char       *transname;
+    const char *transname;
 
 #ifdef TRANS_CLIENT
 
@@ -1658,7 +1648,7 @@ static LOCALtrans2dev LOCALtrans2devtab[] = {
 
 #define NUMTRANSPORTS  (sizeof(LOCALtrans2devtab)/sizeof(LOCALtrans2dev))
 
-static char    *XLOCAL=NULL;
+static const char      *XLOCAL=NULL;
 static char    *workingXLOCAL=NULL;
 static char    *freeXLOCAL=NULL;
 
@@ -1673,24 +1663,20 @@ static  char    *freeXLOCAL=NULL;
 #endif
 
 static void
-TRANS(LocalInitTransports)(char *protocol)
+TRANS(LocalInitTransports)(const char *protocol)
 
 {
-    PRMSG(3,"LocalInitTransports(%s)\n", protocol, 0,0 );
+    prmsg(3,"LocalInitTransports(%s)\n", protocol);
 
     if( strcmp(protocol,"local") && strcmp(protocol,"LOCAL") )
     {
-       workingXLOCAL=freeXLOCAL=(char *)xalloc (strlen (protocol) + 1);
-       if (workingXLOCAL)
-           strcpy (workingXLOCAL, protocol);
+       workingXLOCAL = freeXLOCAL = strdup (protocol);
     }
     else {
        XLOCAL=(char *)getenv("XLOCAL");
        if(XLOCAL==NULL)
            XLOCAL=DEF_XLOCAL;
-       workingXLOCAL=freeXLOCAL=(char *)xalloc (strlen (XLOCAL) + 1);
-       if (workingXLOCAL)
-           strcpy (workingXLOCAL, XLOCAL);
+       workingXLOCAL = freeXLOCAL = strdup (XLOCAL);
     }
 }
 
@@ -1698,8 +1684,8 @@ static void
 TRANS(LocalEndTransports)(void)
 
 {
-    PRMSG(3,"LocalEndTransports()\n", 0,0,0 );
-    xfree(freeXLOCAL);
+    prmsg(3,"LocalEndTransports()\n");
+    free(freeXLOCAL);
 }
 
 #define TYPEBUFSIZE    32
@@ -1713,7 +1699,7 @@ TRANS(LocalGetNextTransport)(void)
     int        i,j;
     char       *typetocheck;
     char       typebuf[TYPEBUFSIZE];
-    PRMSG(3,"LocalGetNextTransport()\n", 0,0,0 );
+    prmsg(3,"LocalGetNextTransport()\n");
 
     while(1)
     {
@@ -1793,7 +1779,7 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
     XtransConnInfo ciptr;
     int index;
 
-    PRMSG(3,"LocalOpenClient()\n", 0,0,0 );
+    prmsg(3,"LocalOpenClient()\n");
 
     /*
      * Make sure 'host' is really local.  If not, we return failure.
@@ -1805,9 +1791,9 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
 
     if (strcmp (host, "unix") != 0 && !HostReallyLocal (host))
     {
-       PRMSG (1,
+       prmsg (1,
           "LocalOpenClient: Cannot connect to non-local host %s\n",
-              host, 0, 0);
+              host);
        return NULL;
     }
 
@@ -1824,10 +1810,10 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
      */
 #endif /* X11_t */
 
-    if( (ciptr=(XtransConnInfo)xcalloc(1,sizeof(struct _XtransConnInfo))) == NULL )
+    if( (ciptr = calloc(1,sizeof(struct _XtransConnInfo))) == NULL )
     {
-       PRMSG(1,"LocalOpenClient: calloc(1,%d) failed\n",
-             sizeof(struct _XtransConnInfo),0,0 );
+       prmsg(1,"LocalOpenClient: calloc(1,%lu) failed\n",
+             sizeof(struct _XtransConnInfo));
        return NULL;
     }
 
@@ -1849,14 +1835,13 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
            break;
        case XTRANS_OPEN_COTS_SERVER:
        case XTRANS_OPEN_CLTS_SERVER:
-           PRMSG(1,
-                 "LocalOpenClient: Should not be opening a server with this function\n",
-                 0,0,0);
+           prmsg(1,
+                 "LocalOpenClient: Should not be opening a server with this function\n");
            break;
        default:
-           PRMSG(1,
+           prmsg(1,
                  "LocalOpenClient: Unknown Open type %d\n",
-                 type, 0,0 );
+                 type);
        }
        if( ciptr->fd >= 0 )
            break;
@@ -1866,7 +1851,7 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
 
     if( ciptr->fd < 0 )
     {
-       xfree(ciptr);
+       free(ciptr);
        return NULL;
     }
 
@@ -1882,13 +1867,13 @@ TRANS(LocalOpenClient)(int type, char *protocol, char *host, char *port)
 #ifdef TRANS_SERVER
 
 static XtransConnInfo
-TRANS(LocalOpenServer)(int type, char *protocol, char *host, char *port)
+TRANS(LocalOpenServer)(int type, char *protocol, char *host _X_UNUSED, char *port)
 
 {
     int        i;
     XtransConnInfo ciptr;
 
-    PRMSG(2,"LocalOpenServer(%d,%s,%s)\n", type, protocol, port);
+    prmsg(2,"LocalOpenServer(%d,%s,%s)\n", type, protocol, port);
 
 #if defined(X11_t)
     /*
@@ -1899,10 +1884,10 @@ TRANS(LocalOpenServer)(int type, char *protocol, char *host, char *port)
      */
 #endif /* X11_t */
 
-    if( (ciptr=(XtransConnInfo)xcalloc(1,sizeof(struct _XtransConnInfo))) == NULL )
+    if( (ciptr = calloc(1,sizeof(struct _XtransConnInfo))) == NULL )
     {
-       PRMSG(1,"LocalOpenServer: calloc(1,%d) failed\n",
-             sizeof(struct _XtransConnInfo),0,0 );
+       prmsg(1,"LocalOpenServer: calloc(1,%lu) failed\n",
+             sizeof(struct _XtransConnInfo));
        return NULL;
     }
 
@@ -1914,9 +1899,8 @@ TRANS(LocalOpenServer)(int type, char *protocol, char *host, char *port)
        {
        case XTRANS_OPEN_COTS_CLIENT:
        case XTRANS_OPEN_CLTS_CLIENT:
-           PRMSG(1,
-                 "LocalOpenServer: Should not be opening a client with this function\n",
-                 0,0,0);
+           prmsg(1,
+                 "LocalOpenServer: Should not be opening a client with this function\n");
            break;
        case XTRANS_OPEN_COTS_SERVER:
            ciptr->fd=LOCALtrans2devtab[i].devcotsopenserver(ciptr,port);
@@ -1925,8 +1909,8 @@ TRANS(LocalOpenServer)(int type, char *protocol, char *host, char *port)
            ciptr->fd=LOCALtrans2devtab[i].devcltsopenserver(ciptr,port);
            break;
        default:
-           PRMSG(1,"LocalOpenServer: Unknown Open type %d\n",
-                 type ,0,0);
+           prmsg(1,"LocalOpenServer: Unknown Open type %d\n",
+                 type );
        }
        if( ciptr->fd >= 0 ) {
            ciptr->priv=(char *)&LOCALtrans2devtab[i];
@@ -1936,7 +1920,7 @@ TRANS(LocalOpenServer)(int type, char *protocol, char *host, char *port)
        }
     }
 
-    xfree(ciptr);
+    free(ciptr);
     return NULL;
 }
 
@@ -1952,12 +1936,12 @@ TRANS(LocalReopenServer)(int type, int index, int fd, char *port)
     XtransConnInfo ciptr;
     int stat = 0;
 
-    PRMSG(2,"LocalReopenServer(%d,%d,%d)\n", type, index, fd);
+    prmsg(2,"LocalReopenServer(%d,%d,%d)\n", type, index, fd);
 
-    if( (ciptr=(XtransConnInfo)xcalloc(1,sizeof(struct _XtransConnInfo))) == NULL )
+    if( (ciptr = calloc(1,sizeof(struct _XtransConnInfo))) == NULL )
     {
-       PRMSG(1,"LocalReopenServer: calloc(1,%d) failed\n",
-             sizeof(struct _XtransConnInfo),0,0 );
+       prmsg(1,"LocalReopenServer: calloc(1,%lu) failed\n",
+             sizeof(struct _XtransConnInfo));
        return NULL;
     }
 
@@ -1972,8 +1956,8 @@ TRANS(LocalReopenServer)(int type, int index, int fd, char *port)
        stat = LOCALtrans2devtab[index].devcltsreopenserver(ciptr,fd,port);
        break;
     default:
-       PRMSG(1,"LocalReopenServer: Unknown Open type %d\n",
-         type ,0,0);
+       prmsg(1,"LocalReopenServer: Unknown Open type %d\n",
+         type );
     }
 
     if( stat > 0 ) {
@@ -1983,7 +1967,7 @@ TRANS(LocalReopenServer)(int type, int index, int fd, char *port)
        return ciptr;
     }
 
-    xfree(ciptr);
+    free(ciptr);
     return NULL;
 }
 
@@ -1998,11 +1982,11 @@ TRANS(LocalReopenServer)(int type, int index, int fd, char *port)
 #ifdef TRANS_CLIENT
 
 static XtransConnInfo
-TRANS(LocalOpenCOTSClient)(Xtransport *thistrans, char *protocol, 
+TRANS(LocalOpenCOTSClient)(Xtransport *thistrans _X_UNUSED, char *protocol,
                           char *host, char *port)
 
 {
-    PRMSG(2,"LocalOpenCOTSClient(%s,%s,%s)\n",protocol,host,port);
+    prmsg(2,"LocalOpenCOTSClient(%s,%s,%s)\n",protocol,host,port);
 
     return TRANS(LocalOpenClient)(XTRANS_OPEN_COTS_CLIENT, protocol, host, port);
 }
@@ -2013,7 +1997,7 @@ TRANS(LocalOpenCOTSClient)(Xtransport *thistrans, char *protocol,
 #ifdef TRANS_SERVER
 
 static XtransConnInfo
-TRANS(LocalOpenCOTSServer)(Xtransport *thistrans, char *protocol, 
+TRANS(LocalOpenCOTSServer)(Xtransport *thistrans, char *protocol,
                           char *host, char *port)
 
 {
@@ -2021,7 +2005,7 @@ TRANS(LocalOpenCOTSServer)(Xtransport *thistrans, char *protocol,
     int found = 0;
     char typebuf[TYPEBUFSIZE];
 
-    PRMSG(2,"LocalOpenCOTSServer(%s,%s,%s)\n",protocol,host,port);
+    prmsg(2,"LocalOpenCOTSServer(%s,%s,%s)\n",protocol,host,port);
 
     /* Check if this local type is in the XLOCAL list */
     TRANS(LocalInitTransports)("local");
@@ -2043,7 +2027,7 @@ TRANS(LocalOpenCOTSServer)(Xtransport *thistrans, char *protocol,
     TRANS(LocalEndTransports)();
 
     if (!found) {
-       PRMSG(3,"LocalOpenCOTSServer: disabling %s\n",thistrans->TransName,0,0);
+       prmsg(3,"LocalOpenCOTSServer: disabling %s\n",thistrans->TransName);
        thistrans->flags |= TRANS_DISABLED;
        return NULL;
     }
@@ -2057,11 +2041,11 @@ TRANS(LocalOpenCOTSServer)(Xtransport *thistrans, char *protocol,
 #ifdef TRANS_CLIENT
 
 static XtransConnInfo
-TRANS(LocalOpenCLTSClient)(Xtransport *thistrans, char *protocol, 
+TRANS(LocalOpenCLTSClient)(Xtransport *thistrans _X_UNUSED, char *protocol,
                           char *host, char *port)
 
 {
-    PRMSG(2,"LocalOpenCLTSClient(%s,%s,%s)\n",protocol,host,port);
+    prmsg(2,"LocalOpenCLTSClient(%s,%s,%s)\n",protocol,host,port);
 
     return TRANS(LocalOpenClient)(XTRANS_OPEN_CLTS_CLIENT, protocol, host, port);
 }
@@ -2072,11 +2056,11 @@ TRANS(LocalOpenCLTSClient)(Xtransport *thistrans, char *protocol,
 #ifdef TRANS_SERVER
 
 static XtransConnInfo
-TRANS(LocalOpenCLTSServer)(Xtransport *thistrans, char *protocol, 
+TRANS(LocalOpenCLTSServer)(Xtransport *thistrans _X_UNUSED, char *protocol,
                           char *host, char *port)
 
 {
-    PRMSG(2,"LocalOpenCLTSServer(%s,%s,%s)\n",protocol,host,port);
+    prmsg(2,"LocalOpenCLTSServer(%s,%s,%s)\n",protocol,host,port);
 
     return TRANS(LocalOpenServer)(XTRANS_OPEN_CLTS_SERVER, protocol, host, port);
 }
@@ -2092,7 +2076,7 @@ TRANS(LocalReopenCOTSServer)(Xtransport *thistrans, int fd, char *port)
 {
     int index;
 
-    PRMSG(2,"LocalReopenCOTSServer(%d,%s)\n", fd, port, 0);
+    prmsg(2,"LocalReopenCOTSServer(%d,%s)\n", fd, port);
 
     for(index=1;index<NUMTRANSPORTS;index++)
     {
@@ -2116,7 +2100,7 @@ TRANS(LocalReopenCLTSServer)(Xtransport *thistrans, int fd, char *port)
 {
     int index;
 
-    PRMSG(2,"LocalReopenCLTSServer(%d,%s)\n", fd, port, 0);
+    prmsg(2,"LocalReopenCLTSServer(%d,%s)\n", fd, port);
 
     for(index=1;index<NUMTRANSPORTS;index++)
     {
@@ -2142,7 +2126,7 @@ static int
 TRANS(LocalSetOption)(XtransConnInfo ciptr, int option, int arg)
 
 {
-    PRMSG(2,"LocalSetOption(%d,%d,%d)\n",ciptr->fd,option,arg);
+    prmsg(2,"LocalSetOption(%d,%d,%d)\n",ciptr->fd,option,arg);
 
     return -1;
 }
@@ -2151,10 +2135,10 @@ TRANS(LocalSetOption)(XtransConnInfo ciptr, int option, int arg)
 #ifdef TRANS_SERVER
 
 static int
-TRANS(LocalCreateListener)(XtransConnInfo ciptr, char *port, unsigned int flags)
+TRANS(LocalCreateListener)(XtransConnInfo ciptr, char *port, unsigned int flags _X_UNUSED)
 
 {
-    PRMSG(2,"LocalCreateListener(%x->%d,%s)\n",ciptr,ciptr->fd,port);
+    prmsg(2,"LocalCreateListener(%p->%d,%s)\n",ciptr,ciptr->fd,port);
 
     return 0;
 }
@@ -2164,8 +2148,8 @@ TRANS(LocalResetListener)(XtransConnInfo ciptr)
 
 {
     LOCALtrans2dev     *transptr;
-    
-    PRMSG(2,"LocalResetListener(%x)\n",ciptr,0,0);
+
+    prmsg(2,"LocalResetListener(%p)\n",ciptr);
 
     transptr=(LOCALtrans2dev *)ciptr->priv;
     if (transptr->devreset != NULL) {
@@ -2182,14 +2166,14 @@ TRANS(LocalAccept)(XtransConnInfo ciptr, int *status)
     XtransConnInfo     newciptr;
     LOCALtrans2dev     *transptr;
 
-    PRMSG(2,"LocalAccept(%x->%d)\n", ciptr, ciptr->fd,0);
+    prmsg(2,"LocalAccept(%p->%d)\n", ciptr, ciptr->fd);
 
     transptr=(LOCALtrans2dev *)ciptr->priv;
 
-    if( (newciptr=(XtransConnInfo)xcalloc(1,sizeof(struct _XtransConnInfo)))==NULL )
+    if( (newciptr = calloc(1,sizeof(struct _XtransConnInfo)))==NULL )
     {
-       PRMSG(1,"LocalAccept: calloc(1,%d) failed\n",
-             sizeof(struct _XtransConnInfo),0,0 );
+       prmsg(1,"LocalAccept: calloc(1,%lu) failed\n",
+             sizeof(struct _XtransConnInfo));
        *status = TRANS_ACCEPT_BAD_MALLOC;
        return NULL;
     }
@@ -2198,7 +2182,7 @@ TRANS(LocalAccept)(XtransConnInfo ciptr, int *status)
 
     if( newciptr->fd < 0 )
     {
-       xfree(newciptr);
+       free(newciptr);
        return NULL;
     }
 
@@ -2216,10 +2200,10 @@ TRANS(LocalAccept)(XtransConnInfo ciptr, int *status)
 #ifdef TRANS_CLIENT
 
 static int
-TRANS(LocalConnect)(XtransConnInfo ciptr, char *host, char *port)
+TRANS(LocalConnect)(XtransConnInfo ciptr, char *host _X_UNUSED, char *port)
 
 {
-    PRMSG(2,"LocalConnect(%x->%d,%s)\n", ciptr, ciptr->fd, port);
+    prmsg(2,"LocalConnect(%p->%d,%s)\n", ciptr, ciptr->fd, port);
 
     return 0;
 }
@@ -2231,7 +2215,7 @@ static int
 TRANS(LocalBytesReadable)(XtransConnInfo ciptr, BytesReadable_t *pend )
 
 {
-    PRMSG(2,"LocalBytesReadable(%x->%d,%x)\n", ciptr, ciptr->fd, pend);
+    prmsg(2,"LocalBytesReadable(%p->%d,%p)\n", ciptr, ciptr->fd, pend);
 
 #if defined(SCO325)
     return ioctl(ciptr->fd, I_NREAD, (char *)pend);
@@ -2244,7 +2228,7 @@ static int
 TRANS(LocalRead)(XtransConnInfo ciptr, char *buf, int size)
 
 {
-    PRMSG(2,"LocalRead(%d,%x,%d)\n", ciptr->fd, buf, size );
+    prmsg(2,"LocalRead(%d,%p,%d)\n", ciptr->fd, buf, size );
 
     return read(ciptr->fd,buf,size);
 }
@@ -2253,7 +2237,7 @@ static int
 TRANS(LocalWrite)(XtransConnInfo ciptr, char *buf, int size)
 
 {
-    PRMSG(2,"LocalWrite(%d,%x,%d)\n", ciptr->fd, buf, size );
+    prmsg(2,"LocalWrite(%d,%p,%d)\n", ciptr->fd, buf, size );
 
     return write(ciptr->fd,buf,size);
 }
@@ -2262,7 +2246,7 @@ static int
 TRANS(LocalReadv)(XtransConnInfo ciptr, struct iovec *buf, int size)
 
 {
-    PRMSG(2,"LocalReadv(%d,%x,%d)\n", ciptr->fd, buf, size );
+    prmsg(2,"LocalReadv(%d,%p,%d)\n", ciptr->fd, buf, size );
 
     return READV(ciptr,buf,size);
 }
@@ -2271,7 +2255,7 @@ static int
 TRANS(LocalWritev)(XtransConnInfo ciptr, struct iovec *buf, int size)
 
 {
-    PRMSG(2,"LocalWritev(%d,%x,%d)\n", ciptr->fd, buf, size );
+    prmsg(2,"LocalWritev(%d,%p,%d)\n", ciptr->fd, buf, size );
 
     return WRITEV(ciptr,buf,size);
 }
@@ -2280,7 +2264,7 @@ static int
 TRANS(LocalDisconnect)(XtransConnInfo ciptr)
 
 {
-    PRMSG(2,"LocalDisconnect(%x->%d)\n", ciptr, ciptr->fd, 0);
+    prmsg(2,"LocalDisconnect(%p->%d)\n", ciptr, ciptr->fd);
 
     return 0;
 }
@@ -2292,7 +2276,7 @@ TRANS(LocalClose)(XtransConnInfo ciptr)
     struct sockaddr_un      *sockname=(struct sockaddr_un *) ciptr->addr;
     int        ret;
 
-    PRMSG(2,"LocalClose(%x->%d)\n", ciptr, ciptr->fd ,0);
+    prmsg(2,"LocalClose(%p->%d)\n", ciptr, ciptr->fd );
 
     ret=close(ciptr->fd);
 
@@ -2314,7 +2298,7 @@ TRANS(LocalCloseForCloning)(XtransConnInfo ciptr)
 {
     int ret;
 
-    PRMSG(2,"LocalCloseForCloning(%x->%d)\n", ciptr, ciptr->fd ,0);
+    prmsg(2,"LocalCloseForCloning(%p->%d)\n", ciptr, ciptr->fd );
 
     /* Don't unlink path */
 
@@ -2328,22 +2312,22 @@ TRANS(LocalCloseForCloning)(XtransConnInfo ciptr)
  * MakeAllCOTSServerListeners() will go through the entire Xtransports[]
  * array defined in Xtrans.c and try to OpenCOTSServer() for each entry.
  * We will add duplicate entries to that table so that the OpenCOTSServer()
- * function will get called once for each type of local transport. 
+ * function will get called once for each type of local transport.
  *
  * The TransName is in lowercase, so it will never match during a normal
  * call to SelectTransport() in Xtrans.c.
  */
 
 #ifdef TRANS_SERVER
-static char * local_aliases[] = {
+static const char * local_aliases[] = {
 # ifdef LOCAL_TRANS_PTS
                                   "pts",
 # endif
                                  "named",
 # ifdef sun
                                  "pipe", /* compatibility with Solaris Xlib */
-# endif                                  
-# ifdef LOCAL_TRANS_SCO                                  
+# endif
+# ifdef LOCAL_TRANS_SCO
                                  "sco",
 # endif
                                  NULL };