Last minute clean up for SunOS...

scrappy · scrappy · commit 6a9627e79227 · 1998-03-02T05:42:15.000Z
From: t-ishii@sra.co.jp
diff --git a/src/backend/libpq/pqcomm.c b/src/backend/libpq/pqcomm.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/libpq/pqcomm.c,v 1.39 1998/02/26 04:31:53 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/libpq/pqcomm.c,v 1.40 1998/03/02 05:41:53 scrappy Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -564,8 +564,8 @@ static char sock_path[MAXPGPATH + 1] = "";
  * Shutdown routine for backend connection
  * If a Unix socket is used for communication, explicitly close it.
  */
-static void
-do_unlink()
+void
+StreamDoUnlink()
 {
 	if (sock_path[0])
 		unlink(sock_path);
@@ -645,7 +645,6 @@ StreamServerPort(char *hostName, short portName, int *fdP)
 	if (family == AF_UNIX)
 	{
 		chmod(sock_path, 0777);
-		atexit(do_unlink);
 	}
 	return (STATUS_OK);
 }
diff --git a/src/backend/storage/ipc/ipc.c b/src/backend/storage/ipc/ipc.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipc.c,v 1.19 1998/02/26 04:35:38 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/storage/ipc/ipc.c,v 1.20 1998/03/02 05:41:55 scrappy Exp $
  *
  * NOTES
  *
@@ -38,6 +38,7 @@
 #include <sys/sem.h>
 #include <sys/shm.h>
 #include "utils/memutils.h"
+#include "libpq/libpq.h"
 
 #if defined(sparc_solaris)
 #include <string.h>
@@ -136,6 +137,7 @@ exitpg(int code)
 	for (i = onexit_index - 1; i >= 0; --i)
 		(*onexit_list[i].function) (code, onexit_list[i].arg);
 
+ 	StreamDoUnlink();
 	exit(code);
 }
 
diff --git a/src/include/libpq/libpq.h b/src/include/libpq/libpq.h
@@ -6,7 +6,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: libpq.h,v 1.12 1998/02/26 04:41:51 momjian Exp $
+ * $Id: libpq.h,v 1.13 1998/03/02 05:42:06 scrappy Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -282,5 +282,6 @@ extern void pq_async_notify(void);
 extern int	StreamServerPort(char *hostName, short portName, int *fdP);
 extern int	StreamConnection(int server_fd, Port *port);
 extern void StreamClose(int sock);
+extern void StreamDoUnlink(void);
 
 #endif							/* LIBPQ_H */
diff --git a/src/include/libpq/pqcomm.h b/src/include/libpq/pqcomm.h
@@ -6,7 +6,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: pqcomm.h,v 1.23 1998/02/26 04:41:53 momjian Exp $
+ * $Id: pqcomm.h,v 1.24 1998/03/02 05:42:15 scrappy Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -35,8 +35,9 @@ typedef union SockAddr
 /* Configure the UNIX socket address for the well known port. */
 
 #define UNIXSOCK_PATH(sun,port) \
-	(sprintf((sun).sun_path, "/tmp/.s.PGSQL.%d", (port)) + \
-		offsetof(struct sockaddr_un, sun_path))
+ 	(sprintf((sun).sun_path, "/tmp/.s.PGSQL.%d", (port)), \
+ 	 strlen((sun).sun_path)+ offsetof(struct sockaddr_un, sun_path))
+
 /*
  *		We do this because sun_len is in BSD's struct, while others don't.
  *		We never actually set BSD's sun_len, and I can't think of a

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`*`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $Header: /cvsroot/pgsql/src/backend/libpq/pqcomm.c,v 1.39 1998/02/26 04:31:53 momjian Exp $`
	`10`	`+ * $Header: /cvsroot/pgsql/src/backend/libpq/pqcomm.c,v 1.40 1998/03/02 05:41:53 scrappy Exp $`
`11`	`11`	`*`
`12`	`12`	`*-------------------------------------------------------------------------`
`13`	`13`	`*/`
`@@ -564,8 +564,8 @@ static char sock_path[MAXPGPATH + 1] = "";`
`564`	`564`	`* Shutdown routine for backend connection`
`565`	`565`	`* If a Unix socket is used for communication, explicitly close it.`
`566`	`566`	`*/`
`567`		`-static void`
`568`		`-do_unlink()`
	`567`	`+void`
	`568`	`+StreamDoUnlink()`
`569`	`569`	`{`
`570`	`570`	`if (sock_path[0])`
`571`	`571`	`unlink(sock_path);`
`@@ -645,7 +645,6 @@ StreamServerPort(char hostName, short portName, int fdP)`
`645`	`645`	`if (family == AF_UNIX)`
`646`	`646`	`{`
`647`	`647`	`chmod(sock_path, 0777);`
`648`		`- atexit(do_unlink);`
`649`	`648`	`}`
`650`	`649`	`return (STATUS_OK);`
`651`	`650`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`	`*`
`7`	`7`	`* Copyright (c) 1994, Regents of the University of California`
`8`	`8`	`*`
`9`		`- * $Id: libpq.h,v 1.12 1998/02/26 04:41:51 momjian Exp $`
	`9`	`+ * $Id: libpq.h,v 1.13 1998/03/02 05:42:06 scrappy Exp $`
`10`	`10`	`*`
`11`	`11`	`*-------------------------------------------------------------------------`
`12`	`12`	`*/`
`@@ -282,5 +282,6 @@ extern void pq_async_notify(void);`
`282`	`282`	`extern int StreamServerPort(char hostName, short portName, int fdP);`
`283`	`283`	`extern int StreamConnection(int server_fd, Port *port);`
`284`	`284`	`extern void StreamClose(int sock);`
	`285`	`+extern void StreamDoUnlink(void);`
`285`	`286`
`286`	`287`	`#endif /* LIBPQ_H */`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`	`*`
`7`	`7`	`* Copyright (c) 1994, Regents of the University of California`
`8`	`8`	`*`
`9`		`- * $Id: pqcomm.h,v 1.23 1998/02/26 04:41:53 momjian Exp $`
	`9`	`+ * $Id: pqcomm.h,v 1.24 1998/03/02 05:42:15 scrappy Exp $`
`10`	`10`	`*`
`11`	`11`	`*-------------------------------------------------------------------------`
`12`	`12`	`*/`
`@@ -35,8 +35,9 @@ typedef union SockAddr`
`35`	`35`	`/* Configure the UNIX socket address for the well known port. */`
`36`	`36`
`37`	`37`	`#define UNIXSOCK_PATH(sun,port) \`
`38`		`- (sprintf((sun).sun_path, "/tmp/.s.PGSQL.%d", (port)) + \`
`39`		`- offsetof(struct sockaddr_un, sun_path))`
	`38`	`+ (sprintf((sun).sun_path, "/tmp/.s.PGSQL.%d", (port)), \`
	`39`	`+ strlen((sun).sun_path)+ offsetof(struct sockaddr_un, sun_path))`
	`40`	`+`
`40`	`41`	`/*`
`41`	`42`	`* We do this because sun_len is in BSD's struct, while others don't.`
`42`	`43`	`* We never actually set BSD's sun_len, and I can't think of a`