Fix additional clang finds.

diff --git a/libmemcached/auto.cc b/libmemcached/auto.cc
index 9a044faad6f8d0d3f68111cbdd5c0bd87bb61d78..d0b7dd4813bb3116738f5713d4e048b68496954b 100644 (file)
--- a/libmemcached/auto.cc
+++ b/libmemcached/auto.cc
@@ -75,9 +75,9 @@ static memcached_return_t text_incr_decr(memcached_st *ptr,
 
   rc= memcached_response(instance, buffer, MEMCACHED_DEFAULT_COMMAND_SIZE, NULL);
 
-  if (rc == MEMCACHED_SUCCESS)
+  if (rc != MEMCACHED_SUCCESS)
   {
-    return MEMCACHED_SUCCESS;
+    return memcached_set_error(*instance, rc, MEMCACHED_AT);
   }
 
   /*

diff --git a/libmemcached/protocol/cache.c b/libmemcached/protocol/cache.c
index 52904df4a7ff3e70fbc8268b908bcb76d6e7b4d9..f92fdec106cdac5f08b74eec8223eddb97a5505a 100644 (file)
--- a/libmemcached/protocol/cache.c
+++ b/libmemcached/protocol/cache.c
@@ -56,7 +56,9 @@ cache_t* cache_create(const char *name, size_t bufsize, size_t align,
                       cache_constructor_t* constructor,
                       cache_destructor_t* destructor) {
     cache_t* ret = calloc(1, sizeof(cache_t));
-    char* nm = strdup(name);
+    size_t name_length= strlen(name);
+    char* nm= calloc(1, (sizeof(char) * name_length) +1);
+    memcpy(nm, name, name_length);
     void** ptr = calloc(initial_pool_size, bufsize);
     if (ret == NULL || nm == NULL || ptr == NULL ||
         pthread_mutex_init(&ret->mutex, NULL) == -1) {

diff --git a/libmemcached/server.h b/libmemcached/server.h
index 3afb5202f6eb95dcabf55cf0a73f688dd931ec13..3ca1df32eddec373f033ac71261bc78937031f29 100644 (file)
--- a/libmemcached/server.h
+++ b/libmemcached/server.h
@@ -42,6 +42,12 @@
 #include <netdb.h>
 #endif
 
+#ifdef NI_MAXHOST
+#define MEMCACHED_NI_MAXHOST NI_MAXHOST
+#else
+#define MEMCACHED_NI_MAXHOST 1025
+#endif
+
 enum memcached_server_state_t {
   MEMCACHED_SERVER_STATE_NEW, // fd == -1, no address lookup has been done
   MEMCACHED_SERVER_STATE_ADDRINFO, // ADDRRESS information has been gathered
@@ -87,7 +93,7 @@ struct memcached_server_st {
   struct memcached_error_t *error_messages;
   char read_buffer[MEMCACHED_MAX_BUFFER];
   char write_buffer[MEMCACHED_MAX_BUFFER];
-  char hostname[NI_MAXHOST];
+  char hostname[MEMCACHED_NI_MAXHOST];
 };
 
 

diff --git a/tests/mem_functions.cc b/tests/mem_functions.cc
index dd46c82f4e162a7eb138d3202d9340e26bb7b48a..c0dde998b83d3a4cb3766ce04348eb9c2babe91d 100644 (file)
--- a/tests/mem_functions.cc
+++ b/tests/mem_functions.cc
@@ -853,7 +853,7 @@ static test_return_t bad_key_test(memcached_st *memc)
       test_zero(string_length);
       test_false(string);
     }
-    test_compare(before_query_id, memcached_query_id(memc_clone) +1);
+    test_compare(before_query_id +1, memcached_query_id(memc_clone));
 
     query_id= memcached_query_id(memc_clone);
     test_compare(MEMCACHED_SUCCESS,