From: Peter Eisentraut <peter_e@gmx.net>
Date: Tue, 23 Oct 2018 12:35:59 +0000 (+0200)
Subject: Drop const cast from dlsym() calls
X-Git-Tag: REL_12_BETA1~1353
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e6f5d1accd3ae170a0baa632bfe3396bdd9b6e92;p=thirdparty%2Fpostgresql.git

Drop const cast from dlsym() calls

This workaround might be obsolete.  We'll see if those "older
platforms" mentioned in the comment are still around.

Discussion: https://www.postgresql.org/message-id/08adbe4e-38f8-2c73-55f0-591392371687%402ndquadrant.com
---

diff --git a/src/backend/utils/fmgr/dfmgr.c b/src/backend/utils/fmgr/dfmgr.c
index 4a5cc7cfc7f..04c8aa8eef0 100644
--- a/src/backend/utils/fmgr/dfmgr.c
+++ b/src/backend/utils/fmgr/dfmgr.c
@@ -121,12 +121,8 @@ load_external_function(const char *filename, const char *funcname,
 	if (filehandle)
 		*filehandle = lib_handle;
 
-	/*
-	 * Look up the function within the library.  According to POSIX dlsym()
-	 * should declare its second argument as "const char *", but older
-	 * platforms might not, so for the time being we just cast away const.
-	 */
-	retval = (PGFunction) dlsym(lib_handle, (char *) funcname);
+	/* Look up the function within the library. */
+	retval = (PGFunction) dlsym(lib_handle, funcname);
 
 	if (retval == NULL && signalNotFound)
 		ereport(ERROR,
@@ -174,8 +170,7 @@ load_file(const char *filename, bool restricted)
 PGFunction
 lookup_external_function(void *filehandle, const char *funcname)
 {
-	/* as above, cast away const for the time being */
-	return (PGFunction) dlsym(filehandle, (char *) funcname);
+	return (PGFunction) dlsym(filehandle, funcname);
 }