Merge pull request #1091 from redis/ssl-error-ub-fix
Fix some undefined behavior
This commit is contained in:
commit
df8b74d69e
@ -56,7 +56,7 @@ int main (int argc, char **argv) {
|
|||||||
const char *caCert = argc > 5 ? argv[6] : NULL;
|
const char *caCert = argc > 5 ? argv[6] : NULL;
|
||||||
|
|
||||||
redisSSLContext *ssl;
|
redisSSLContext *ssl;
|
||||||
redisSSLContextError ssl_error;
|
redisSSLContextError ssl_error = REDIS_SSL_CTX_NONE;
|
||||||
|
|
||||||
redisInitOpenSSL();
|
redisInitOpenSSL();
|
||||||
|
|
||||||
|
@ -12,7 +12,7 @@
|
|||||||
int main(int argc, char **argv) {
|
int main(int argc, char **argv) {
|
||||||
unsigned int j;
|
unsigned int j;
|
||||||
redisSSLContext *ssl;
|
redisSSLContext *ssl;
|
||||||
redisSSLContextError ssl_error;
|
redisSSLContextError ssl_error = REDIS_SSL_CTX_NONE;
|
||||||
redisContext *c;
|
redisContext *c;
|
||||||
redisReply *reply;
|
redisReply *reply;
|
||||||
if (argc < 4) {
|
if (argc < 4) {
|
||||||
@ -27,9 +27,8 @@ int main(int argc, char **argv) {
|
|||||||
|
|
||||||
redisInitOpenSSL();
|
redisInitOpenSSL();
|
||||||
ssl = redisCreateSSLContext(ca, NULL, cert, key, NULL, &ssl_error);
|
ssl = redisCreateSSLContext(ca, NULL, cert, key, NULL, &ssl_error);
|
||||||
if (!ssl) {
|
if (!ssl || ssl_error != REDIS_SSL_CTX_NONE) {
|
||||||
printf("SSL Context error: %s\n",
|
printf("SSL Context error: %s\n", redisSSLContextGetError(ssl_error));
|
||||||
redisSSLContextGetError(ssl_error));
|
|
||||||
exit(1);
|
exit(1);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user