X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=libmemcached%2Fkey.cc;h=39c1455be67b51d02260a7ea02b0492bc86a461b;hb=1bcc4d8c163e2b696d6ce313a6cba423e81173f9;hp=ea98c729f02b92f5fdf187623dba9ec4a37b0ad0;hpb=54b757ae099fb3db5132bd8a84b99a9a85492af1;p=awesomized%2Flibmemcached

diff --git a/libmemcached/key.cc b/libmemcached/key.cc
index ea98c729..39c1455b 100644
--- a/libmemcached/key.cc
+++ b/libmemcached/key.cc
@@ -44,31 +44,21 @@ memcached_return_t memcached_key_test(memcached_st &memc,
 {
   if (keys == NULL or key_length == NULL)
   {
-    return memcached_set_error(memc, MEMCACHED_BAD_KEY_PROVIDED, MEMCACHED_AT);
+    return memcached_set_error(memc, MEMCACHED_BAD_KEY_PROVIDED, MEMCACHED_AT, memcached_literal_param("Key was NULL or length of key was zero."));
   }
 
-  if (not memc.flags.verify_key)
+  // If we don't need to verify the key, or we are using the binary protoocol,
+  // we just check the size of the key
+  if (memc.flags.verify_key == false or memc.flags.binary_protocol == true)
   {
     for (uint32_t x= 0; x < number_of_keys; x++)
     {
-      memcached_return_t rc= memcached_validate_key_length(*(key_length +x), false);
+      // We should set binary key, but the memcached server is broken for
+      // longer keys at the moment.
+      memcached_return_t rc= memcached_validate_key_length(*(key_length +x), false /* memc.flags.binary_protocol */);
       if (memcached_failed(rc))
       {
-        return rc;
-      }
-    }
-
-    return MEMCACHED_SUCCESS;
-  }
-
-  if (memc.flags.binary_protocol)
-  {
-    for (uint32_t x= 0; x < number_of_keys; x++)
-    {
-      memcached_return_t rc= memcached_validate_key_length(*(key_length +x), false);
-      if (memcached_failed(rc))
-      {
-        return rc;
+        return memcached_set_error(memc, rc, MEMCACHED_AT, memcached_literal_param("Key provided was too long."));
       }
     }
 
@@ -80,14 +70,14 @@ memcached_return_t memcached_key_test(memcached_st &memc,
     memcached_return_t rc= memcached_validate_key_length(*(key_length + x), false);
     if (memcached_failed(rc))
     {
-      return rc;
+      return memcached_set_error(memc, rc, MEMCACHED_AT, memcached_literal_param("Key provided was too long."));
     }
  
     for (size_t y= 0; y < *(key_length + x); y++)
     {
       if ((isgraph(keys[x][y])) == 0)
       {
-        return MEMCACHED_BAD_KEY_PROVIDED;
+        return memcached_set_error(memc, MEMCACHED_BAD_KEY_PROVIDED, MEMCACHED_AT, memcached_literal_param("Key provided had invalid character."));
       }
     }
   }