diff options
| author | Peter Powell <petpow@saberuk.com> | 2017-08-26 17:25:20 +0100 |
|---|---|---|
| committer | Peter Powell <petpow@saberuk.com> | 2017-08-26 19:01:27 +0100 |
| commit | 45fc75457b56bb14ad0f7b99909b96404df69c8c (patch) | |
| tree | 6cfff6d7fe565bd8b58346d9f68f982017f9b41b /include | |
| parent | 1f2a8e4db6a4c722de4e090fcd7f710d874de1ed (diff) | |
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.
Diffstat (limited to 'include')
| -rw-r--r-- | include/dynamic.h | 2 |
1 files changed, 0 insertions, 2 deletions
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() |