From d61f0240eefc3c96a36c16a9bf83167eb01dfe94 Mon Sep 17 00:00:00 2001 From: Wilco Baan Hofman Date: Mon, 24 May 2010 19:53:12 +0200 Subject: Fix 'magic' numbers to be strlen(something) Signed-off-by: Jelmer Vernooij --- source4/lib/policy/gp_filesys.c | 5 +++-- source4/lib/policy/gp_ldap.c | 5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) (limited to 'source4/lib/policy') diff --git a/source4/lib/policy/gp_filesys.c b/source4/lib/policy/gp_filesys.c index 6248d401e7..aab1d02c3b 100644 --- a/source4/lib/policy/gp_filesys.c +++ b/source4/lib/policy/gp_filesys.c @@ -346,6 +346,7 @@ NTSTATUS gp_create_gpt(struct gp_context *gp_ctx, const char *name, const char * int rv; int fd; NTSTATUS status; + const char *file_content = "[General]\r\nVersion=0\r\n"; /* Create a forked memory context, as a base for everything here */ mem_ctx = talloc_new(gp_ctx); @@ -388,8 +389,8 @@ NTSTATUS gp_create_gpt(struct gp_context *gp_ctx, const char *name, const char * return NT_STATUS_UNSUCCESSFUL; } - rv = write(fd, "[General]\r\nVersion=0\r\n", 23); - if (rv != 23) { + rv = write(fd, file_content, strlen(file_content)); + if (rv != strlen(file_content)) { DEBUG(0, ("Short write in GPT.INI\n")); talloc_free(mem_ctx); return NT_STATUS_UNSUCCESSFUL; diff --git a/source4/lib/policy/gp_ldap.c b/source4/lib/policy/gp_ldap.c index 0c307085cb..d03ea0430b 100644 --- a/source4/lib/policy/gp_ldap.c +++ b/source4/lib/policy/gp_ldap.c @@ -302,12 +302,13 @@ static NTSTATUS parse_gplink (TALLOC_CTX *mem_ctx, const char *gplink_str, struc int pos; struct gp_link **gplinks; char *buf, *end; + const char *gplink_start = "[LDAP://"; gplinks = talloc_array(mem_ctx, struct gp_link *, 1); gplinks[0] = NULL; /* Assuming every gPLink starts with "[LDAP://" */ - start = 8; + start = strlen(gplink_start); for (pos = start; pos < strlen(gplink_str); pos++) { if (gplink_str[pos] == ';') { @@ -329,7 +330,7 @@ static NTSTATUS parse_gplink (TALLOC_CTX *mem_ctx, const char *gplink_str, struc /* Increment the array index, the string position past the next "[LDAP://", and set the start reference */ idx++; - pos += 9; + pos += strlen(gplink_start)+1; start = pos; } } -- cgit