Add missing PGDLLIMPORT markings
authorMichael Paquier
Tue, 16 Apr 2024 00:38:46 +0000 (09:38 +0900)
committerMichael Paquier
Tue, 16 Apr 2024 00:38:46 +0000 (09:38 +0900)
All backend-side variables should be marked with PGDLLIMPORT, as per
policy introduced in 8ec569479f.  aafc05de1bf5 has forgotten
MyClientSocket, and 05c3980e7f47 LoadedSSL.

These can be spotted with a command like this one (be careful of not
switching __pg_log_level):
src/tools/mark_pgdllimport.pl $(git ls-files src/include/)

Reported-by: Peter Eisentraut
Discussion: https://postgr.es/m/[email protected]

src/include/postmaster/postmaster.h

index 8ce990e8009e090233bc2e652666f6bc6a9631c9..f6152eddf580dcf2fbc75217530d57d258f741c1 100644 (file)
@@ -52,7 +52,7 @@ extern PGDLLIMPORT int postmaster_alive_fds[2];
 
 extern PGDLLIMPORT const char *progname;
 
-extern bool LoadedSSL;
+extern PGDLLIMPORT bool LoadedSSL;
 
 extern void PostmasterMain(int argc, char *argv[]) pg_attribute_noreturn();
 extern void ClosePostmasterPorts(bool am_syslogger);
@@ -74,7 +74,7 @@ extern void pgwin32_register_deadchild_callback(HANDLE procHandle, DWORD procId)
 #endif
 
 /* defined in globals.c */
-extern struct ClientSocket *MyClientSocket;
+extern PGDLLIMPORT struct ClientSocket *MyClientSocket;
 
 /* prototypes for functions in launch_backend.c */
 extern pid_t postmaster_child_launch(BackendType child_type, char *startup_data, size_t startup_data_len, struct ClientSocket *sock);