Merge 2810ddf into a8b2bc5

.travis.yml

@@ -31,11 +31,11 @@ matrix:
       compiler: gcc
       env: CMPLR=clang STATIC=YES
     - sudo: false
-      dist: trusty
+      dist: bionic
       compiler: gcc
       env: WINE=32 TEST=NO STATIC=YES
     - sudo: false
-      dist: trusty
+      dist: bionic
       compiler: gcc
       env: WINE=32 TEST=NO STATIC=NO
     - sudo: false

modules/libcom/src/osi/os/WIN32/osdFindSymbol.c

@@ -5,13 +5,27 @@
 \*************************************************************************/
 /* osi/os/WIN32/osdFindSymbol.c */
 
+/* avoid need to link against psapi.dll
+ * requires windows 7 or later
+ */
+#define NTDDI_VERSION NTDDI_WIN7
 
 #include <windows.h>
+#include <psapi.h>
 
 #define epicsExportSharedSymbols
 #include "epicsFindSymbol.h"
 
-static int epicsLoadErrorCode = 0;
+#ifdef _MSC_VER
+#  define STORE static __declspec( thread )
+#elif __GNUC__
+#  define STORE static __thread
+#else
+#  define STORE static
+#endif
+
+STORE
+int epicsLoadErrorCode = 0;
 
 epicsShareFunc void * epicsLoadLibrary(const char *name)
 {
@@ -28,7 +42,7 @@ epicsShareFunc void * epicsLoadLibrary(const char *name)
 
 epicsShareFunc const char *epicsLoadError(void)
 {
-    static char buffer[100];
+    STORE char buffer[100];
 
     FormatMessage(
         FORMAT_MESSAGE_FROM_SYSTEM,
@@ -42,5 +56,23 @@ epicsShareFunc const char *epicsLoadError(void)
 
 epicsShareFunc void * epicsShareAPI epicsFindSymbol(const char *name)
 {
-    return GetProcAddress(0, name);
+    HMODULE dlls[128];
+    DWORD ndlls=0u, i;
+    void* ret = NULL;
+
+    /* As a handle returned by LoadLibrary() isn't available to us,
+     * try all loaded modules in arbitrary order.
+     */
+    if(K32EnumProcessModules(GetCurrentProcess(), dlls, sizeof(dlls), &ndlls)) {
+        for(i=0; !ret && i<ndlls; i++) {
+            ret = GetProcAddress(dlls[i], name);
+        }
+    }
+    if(!ret) {
+        /* only capturing the last error code,
+         * but what else to do?
+         */
+        epicsLoadErrorCode = GetLastError();
+    }
+    return ret;
 }

modules/libcom/src/osi/os/posix/osdFindSymbol.c

@@ -11,6 +11,12 @@
 #define epicsExportSharedSymbols
 #include "epicsFindSymbol.h"
 
+/* non-POSIX extension available on Linux (glibc at least) and OSX.
+ */
+#ifndef RTLD_DEFAULT
+#  define RTLD_DEFAULT 0
+#endif
+
 epicsShareFunc void * epicsLoadLibrary(const char *name)
 {
     return dlopen(name, RTLD_LAZY | RTLD_GLOBAL);
@@ -23,5 +29,5 @@ epicsShareFunc const char *epicsLoadError(void)
 
 epicsShareFunc void * epicsShareAPI epicsFindSymbol(const char *name)
 {
-    return dlsym(0, name);
+    return dlsym(RTLD_DEFAULT, name);
 }

modules/libcom/test/Makefile

@@ -260,6 +260,16 @@ iocshTest_SRCS += iocshTest.cpp
 TESTS += iocshTest
 TESTFILES += $(wildcard ../iocshTest*.cmd)
 
+TESTPROD_HOST += epicsLoadTest
+epicsLoadTest_SRCS += epicsLoadTest.cpp
+# test linked against static libCom?
+epicsLoadTest_CPPFLAGS_STATIC_YES = -DLINKING_STATIC
+epicsLoadTest_CPPFLAGS += $(epicsLoadTest_CPPFLAGS_STATIC_$(STATIC_BUILD))
+# are dynamic libraries built?
+epicsLoadTest_CPPFLAGS_SHARED_YES = -DBUILDING_SHARED_LIBRARIES
+epicsLoadTest_CPPFLAGS += $(epicsLoadTest_CPPFLAGS_SHARED_$(SHARED_LIBRARIES))
+TESTS += epicsLoadTest
+
 # The testHarness runs all the test programs in a known working order.
 testHarness_SRCS += epicsRunLibComTests.c
 

modules/libcom/test/epicsLoadTest.cpp

@@ -0,0 +1,91 @@
+/*************************************************************************\
+* Copyright (c) 2020 Michael Davidsaver
+* EPICS BASE is distributed subject to a Software License Agreement found
+* in file LICENSE that is included with this distribution.
+\*************************************************************************/
+
+#include <sstream>
+
+#include <stdlib.h>
+
+#include "epicsUnitTest.h"
+#include "testMain.h"
+
+#include "envDefs.h"
+#include "epicsFindSymbol.h"
+#include "epicsThread.h"
+
+namespace {
+
+// lookup a symbol from libCom
+// which this executable is linked against (maybe statically)
+// Doesn't work for static builds on windows
+void loadCom()
+{
+    testDiag("Lookup symbol from Com");
+
+#if defined (_WIN32) && defined(LINKING_STATIC)
+    testTodoBegin("windows static build");
+#endif
+
+    void* ptr = epicsFindSymbol("epicsThreadGetCPUs");
+    testOk(ptr==(void*)&epicsThreadGetCPUs,
+           "%p == %p (epicsThreadGetCPUs) : %s",
+           ptr, (void*)&epicsThreadGetCPUs,
+           epicsLoadError());
+
+    testTodoEnd();
+}
+
+void loadCA()
+{
+    testDiag("Load and lookup symbol from libca");
+
+    std::string libname;
+    {
+        std::ostringstream strm;
+        // running in eg. modules/libcom/test/O.linux-x86_64-debug
+#ifdef _WIN32
+        strm<<"..\\..\\..\\..\\bin\\"<<envGetConfigParamPtr(&EPICS_BUILD_TARGET_ARCH)<<"\\ca.dll";
+#else
+        strm<<"../../../../lib/"<<envGetConfigParamPtr(&EPICS_BUILD_TARGET_ARCH)<<"/";
+#  ifdef __APPLE__
+        strm<<"libca.dylib";
+#  else
+        strm<<"libca.so";
+#  endif
+#endif
+        libname = strm.str();
+    }
+    testDiag("Loading %s", libname.c_str());
+
+    void *ptr = epicsLoadLibrary(libname.c_str());
+    testOk(!!ptr, "Loaded %p : %s", ptr, epicsLoadError());
+
+    ptr = epicsFindSymbol("ca_context_create");
+    testOk(!!ptr, "ca_context_create %p : %s", ptr, epicsLoadError());
+}
+
+} // namespace
+
+MAIN(epicsLoadTest)
+{
+    testPlan(3);
+
+    // reference to ensure linkage when linking statically,
+    // and actually use the result to make extra doubly sure that
+    // this call isn't optimized out by eg. an LTO pass.
+    testDiag("# of CPUs %d", epicsThreadGetCPUs());
+
+#if defined(__rtems__) || defined(vxWorks)
+    testSkip(3, "Target does not implement epicsFindSymbol()");
+#else
+    loadCom();
+#  if defined(BUILDING_SHARED_LIBRARIES)
+    loadCA();
+#  else
+    testSkip(2, "Shared libraries not built");
+#  endif
+#endif
+    return testDone();
+}