From 45fc75457b56bb14ad0f7b99909b96404df69c8c Mon Sep 17 00:00:00 2001 From: Peter Powell Date: Sat, 26 Aug 2017 17:25:20 +0100 Subject: Use DLLManager::RetrieveLastError() on all platforms. This prevents a bug where we send malformed messages to the client when dlerror() returns an error message containing more than one line. This has been observed on macOS but probably will happen on other UNIX systems too. This also fixes a potential problem where dlerror() returns NULL and converting it to std::string causes a crash. I can't see any way that this might happen but it is better to be safe than sorry. --- include/dynamic.h | 2 -- 1 file changed, 2 deletions(-) (limited to 'include') diff --git a/include/dynamic.h b/include/dynamic.h index 5e66ddbb0..bbe89dc7e 100644 --- a/include/dynamic.h +++ b/include/dynamic.h @@ -33,11 +33,9 @@ class CoreExport DLLManager : public classbase */ std::string err; -#ifdef _WIN32 /** Sets the last error string */ void RetrieveLastError(); -#endif public: /** This constructor loads the module using dlopen() -- cgit v1.2.3