From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon, 12 Aug 2002 14:47:46 +0000 (+0000)
Subject: Convert possibly-unsafe sprintf's to snprintf's.
X-Git-Tag: REL9_0_0~17230
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=1ccc2451c62865c691d114c4cb54a9a2a63128d5;p=pg-rex%2Fsyncrep.git

Convert possibly-unsafe sprintf's to snprintf's.
---

diff --git a/src/backend/port/dynloader/freebsd.c b/src/backend/port/dynloader/freebsd.c
index 0f2ec854a3..874eefce36 100644
--- a/src/backend/port/dynloader/freebsd.c
+++ b/src/backend/port/dynloader/freebsd.c
@@ -63,13 +63,15 @@ void *
 BSD44_derived_dlopen(const char *file, int num)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlopen (%s) not supported", file);
+	snprintf(error_message, sizeof(error_message),
+			 "dlopen (%s) not supported", file);
 	return NULL;
 #else
 	void	   *vp;
 
 	if ((vp = dlopen((char *) file, num)) == (void *) NULL)
-		sprintf(error_message, "dlopen '%s' failed. (%s)", file, dlerror());
+		snprintf(error_message, sizeof(error_message),
+				 "dlopen '%s' failed. (%s)", file, dlerror());
 	return vp;
 #endif
 }
@@ -78,7 +80,8 @@ void *
 BSD44_derived_dlsym(void *handle, const char *name)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlsym (%s) failed", name);
+	snprintf(error_message, sizeof(error_message),
+			 "dlsym (%s) failed", name);
 	return NULL;
 #else
 	void	   *vp;
@@ -88,12 +91,13 @@ BSD44_derived_dlsym(void *handle, const char *name)
 
 	if (*name != '_')
 	{
-		sprintf(buf, "_%s", name);
+		snprintf(buf, sizeof(buf), "_%s", name);
 		name = buf;
 	}
 #endif
 	if ((vp = dlsym(handle, (char *) name)) == (void *) NULL)
-		sprintf(error_message, "dlsym (%s) failed", name);
+		snprintf(error_message, sizeof(error_message),
+				 "dlsym (%s) failed", name);
 	return vp;
 #endif
 }
diff --git a/src/backend/port/dynloader/netbsd.c b/src/backend/port/dynloader/netbsd.c
index 881b9711c1..26a4ec72c9 100644
--- a/src/backend/port/dynloader/netbsd.c
+++ b/src/backend/port/dynloader/netbsd.c
@@ -63,13 +63,15 @@ void *
 BSD44_derived_dlopen(const char *file, int num)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlopen (%s) not supported", file);
+	snprintf(error_message, sizeof(error_message),
+			 "dlopen (%s) not supported", file);
 	return NULL;
 #else
 	void	   *vp;
 
 	if ((vp = dlopen((char *) file, num)) == (void *) NULL)
-		sprintf(error_message, "dlopen (%s) failed", file);
+		snprintf(error_message, sizeof(error_message),
+				 "dlopen (%s) failed", file);
 	return vp;
 #endif
 }
@@ -78,7 +80,8 @@ void *
 BSD44_derived_dlsym(void *handle, const char *name)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlsym (%s) failed", name);
+	snprintf(error_message, sizeof(error_message),
+			 "dlsym (%s) failed", name);
 	return NULL;
 #elif defined(__ELF__)
 	return dlsym(handle, name);
@@ -88,11 +91,12 @@ BSD44_derived_dlsym(void *handle, const char *name)
 
 	if (*name != '_')
 	{
-		sprintf(buf, "_%s", name);
+		snprintf(buf, sizeof(buf), "_%s", name);
 		name = buf;
 	}
 	if ((vp = dlsym(handle, (char *) name)) == (void *) NULL)
-		sprintf(error_message, "dlsym (%s) failed", name);
+		snprintf(error_message, sizeof(error_message),
+				 "dlsym (%s) failed", name);
 	return vp;
 #endif
 }
diff --git a/src/backend/port/dynloader/nextstep.c b/src/backend/port/dynloader/nextstep.c
index 98cf5c2f78..c733963607 100644
--- a/src/backend/port/dynloader/nextstep.c
+++ b/src/backend/port/dynloader/nextstep.c
@@ -67,7 +67,7 @@ next_dlsym(void *handle, char *symbol)
 	char		symbuf[1024];
 	unsigned long symref = 0;
 
-	sprintf(symbuf, "_%s", symbol);
+	snprintf(symbuf, sizeof(symbuf), "_%s", symbol);
 	if (!rld_lookup(errorStream, symbuf, &symref))
 		TransferError(errorStream);
 	CloseError(errorStream);
diff --git a/src/backend/port/dynloader/openbsd.c b/src/backend/port/dynloader/openbsd.c
index 881b9711c1..26a4ec72c9 100644
--- a/src/backend/port/dynloader/openbsd.c
+++ b/src/backend/port/dynloader/openbsd.c
@@ -63,13 +63,15 @@ void *
 BSD44_derived_dlopen(const char *file, int num)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlopen (%s) not supported", file);
+	snprintf(error_message, sizeof(error_message),
+			 "dlopen (%s) not supported", file);
 	return NULL;
 #else
 	void	   *vp;
 
 	if ((vp = dlopen((char *) file, num)) == (void *) NULL)
-		sprintf(error_message, "dlopen (%s) failed", file);
+		snprintf(error_message, sizeof(error_message),
+				 "dlopen (%s) failed", file);
 	return vp;
 #endif
 }
@@ -78,7 +80,8 @@ void *
 BSD44_derived_dlsym(void *handle, const char *name)
 {
 #if defined(__mips__)
-	sprintf(error_message, "dlsym (%s) failed", name);
+	snprintf(error_message, sizeof(error_message),
+			 "dlsym (%s) failed", name);
 	return NULL;
 #elif defined(__ELF__)
 	return dlsym(handle, name);
@@ -88,11 +91,12 @@ BSD44_derived_dlsym(void *handle, const char *name)
 
 	if (*name != '_')
 	{
-		sprintf(buf, "_%s", name);
+		snprintf(buf, sizeof(buf), "_%s", name);
 		name = buf;
 	}
 	if ((vp = dlsym(handle, (char *) name)) == (void *) NULL)
-		sprintf(error_message, "dlsym (%s) failed", name);
+		snprintf(error_message, sizeof(error_message),
+				 "dlsym (%s) failed", name);
 	return vp;
 #endif
 }