Avoid dropping const qualifier in casts

Adjusts casts to void dropping const qualifiers. This helps to avoid
mistakes, e.g. modifying string literals.

Also use size_t for length, similar to the upstream php interfaces.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/sp_crypt.c b/src/sp_crypt.c
index a27cc67..6d48554 100644
--- a/src/sp_crypt.c
+++ b/src/sp_crypt.c
@@ -14,11 +14,11 @@ void generate_key(unsigned char *key) {
   PHP_SHA256Init(&ctx);
 
   if (user_agent) {
-    PHP_SHA256Update(&ctx, (unsigned char *)user_agent, strlen(user_agent));
+    PHP_SHA256Update(&ctx, (const unsigned char *)user_agent, strlen(user_agent));
   }
 
   if (env_var) {
-    PHP_SHA256Update(&ctx, (unsigned char *)env_var, strlen(env_var));
+    PHP_SHA256Update(&ctx, (const unsigned char *)env_var, strlen(env_var));
   } else {
     sp_log_warn("cookie_encryption",
                 "The environment variable '%s' "