aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax <post@wickenrode.com>2018-02-14 23:35:04 +0100
committerMax <post@wickenrode.com>2018-02-14 23:35:04 +0100
commitb6e2c43ef3155acd676c12b0d460b312175e4866 (patch)
tree64438a4ef127345fc91ef1bcc1ba3af9d5e750fd
parentfb210cb9d0458d8b3d0fc1c5f7c13d60ae32042d (diff)
downloadsequelpro-b6e2c43ef3155acd676c12b0d460b312175e4866.tar.gz
sequelpro-b6e2c43ef3155acd676c12b0d460b312175e4866.tar.bz2
sequelpro-b6e2c43ef3155acd676c12b0d460b312175e4866.zip
Restore support for Mac OS X < 10.9 that broke in the previous commit
-rw-r--r--Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m29
1 files changed, 27 insertions, 2 deletions
diff --git a/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m b/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
index f6755f60..7738f665 100644
--- a/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
+++ b/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
@@ -37,6 +37,7 @@
#define __STDC_WANT_LIB_EXT1__ 1
#include <string.h>
#include <stdlib.h>
+#include <dlfcn.h>
// Thread flag constant
static pthread_key_t mySQLThreadInitFlagKey;
@@ -54,6 +55,7 @@ const SPMySQLClientFlags SPMySQLConnectionOptions =
const char *SPMySQLSSLPermissibleCiphers = "DHE-RSA-AES256-SHA:AES256-SHA:DHE-RSA-AES128-SHA:AES128-SHA:AES256-RMD:AES128-RMD:DES-CBC3-RMD:DHE-RSA-AES256-RMD:DHE-RSA-AES128-RMD:DHE-RSA-DES-CBC3-RMD:RC4-SHA:RC4-MD5:DES-CBC3-SHA:DES-CBC-SHA:EDH-RSA-DES-CBC3-SHA:EDH-RSA-DES-CBC-SHA";
static void PasswordCallback(MYSQL *mysql, const char *plugin, void (^with_password)(const char *passwd));
+static errno_t LegacyMemsetS(void *ptr, rsize_t ignored, int value, rsize_t count);
@implementation SPMySQLConnection
@@ -773,8 +775,16 @@ static uint64_t _elapsedMicroSecondsSinceAbsoluteTime(uint64_t comparisonTime)
else {
NSLog(@"%s: -getCString:maxLength:encoding: failed for password!", __PRETTY_FUNCTION__);
}
-
- memset_s(cBuffer, cLength, '\0', cLength); //clear password from memory
+
+ // memset_s is 10.9+ only - if we added a link time dependency, SP wouldn't launch on older targets
+ static errno_t (*memsetPtr)(void *, rsize_t, int, rsize_t);
+ static dispatch_once_t findMemsetToken;
+ dispatch_once(&findMemsetToken, ^{
+ memsetPtr = dlsym(RTLD_DEFAULT, "memset_s");
+ if(!memsetPtr) memsetPtr = LegacyMemsetS;
+ });
+
+ memsetPtr(cBuffer, cLength, '\0', cLength); //clear password from memory
free(cBuffer);
}
@@ -1213,3 +1223,18 @@ void PasswordCallback(MYSQL *mysql, const char *plugin, void (^with_password)(co
assert(mysql && mysql->sp_context);
[(SPMySQLConnection *)mysql->sp_context _mysqlConnection:mysql wantsPassword:with_password withPlugin:plugin];
}
+
+/**
+ * This function tries to emulate the important (to us) parts
+ * of memset_s on pre 10.9 systems.
+ *
+ * The implementation is taken from the original memset_s proposal:
+ * http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1381.pdf
+ */
+errno_t LegacyMemsetS(void *s, rsize_t smax, int c, rsize_t n)
+{
+ volatile unsigned char * addr = (volatile unsigned char *)s;
+ while(n--) *addr++ = c;
+
+ return 0;
+}