Remove obvious use of safe_strcpy + safe_strcat -> strlcpy + strlcat.
authorJeremy Allison <jra@samba.org>
Tue, 3 May 2011 20:19:27 +0000 (13:19 -0700)
committerJeremy Allison <jra@samba.org>
Wed, 4 May 2011 19:12:14 +0000 (12:12 -0700)
source3/auth/auth_script.c

index ad973106b1c082fc7256e5ece343cd0984db5813..5691ea14f11a60c1252f90031e624012bc1ed9eb 100644 (file)
@@ -74,32 +74,32 @@ static NTSTATUS script_check_user_credentials(const struct auth_context *auth_co
                return NT_STATUS_NO_MEMORY;
        }
 
-       safe_strcpy( secret_str, user_info->mapped.domain_name, secret_str_len - 1);
-       safe_strcat( secret_str, "\n", secret_str_len - 1);
-       safe_strcat( secret_str, user_info->client.account_name, secret_str_len - 1);
-       safe_strcat( secret_str, "\n", secret_str_len - 1);
+       strlcpy( secret_str, user_info->mapped.domain_name, secret_str_len);
+       strlcat( secret_str, "\n", secret_str_len);
+       strlcat( secret_str, user_info->client.account_name, secret_str_len);
+       strlcat( secret_str, "\n", secret_str_len);
 
        for (i = 0; i < 8; i++) {
                slprintf(&hex_str[i*2], 3, "%02X", auth_context->challenge.data[i]);
        }
-       safe_strcat( secret_str, hex_str, secret_str_len - 1);
-       safe_strcat( secret_str, "\n", secret_str_len - 1);
+       strlcat( secret_str, hex_str, secret_str_len);
+       strlcat( secret_str, "\n", secret_str_len);
 
        if (user_info->password.response.lanman.data) {
                for (i = 0; i < 24; i++) {
                        slprintf(&hex_str[i*2], 3, "%02X", user_info->password.response.lanman.data[i]);
                }
-               safe_strcat( secret_str, hex_str, secret_str_len - 1);
+               strlcat( secret_str, hex_str, secret_str_len);
        }
-       safe_strcat( secret_str, "\n", secret_str_len - 1);
+       strlcat( secret_str, "\n", secret_str_len);
 
        if (user_info->password.response.nt.data) {
                for (i = 0; i < 24; i++) {
                        slprintf(&hex_str[i*2], 3, "%02X", user_info->password.response.nt.data[i]);
                }
-               safe_strcat( secret_str, hex_str, secret_str_len - 1);
+               strlcat( secret_str, hex_str, secret_str_len);
        }
-       safe_strcat( secret_str, "\n", secret_str_len - 1);
+       strlcat( secret_str, "\n", secret_str_len);
 
        DEBUG(10,("script_check_user_credentials: running %s with parameters:\n%s\n",
                script, secret_str ));