Avoid magic constant (32) in allocation. Use sizeof instead

diff --git a/servers/slapd/daemon.c b/servers/slapd/daemon.c
index a0a9f1cbb57547cca4fb36148a964c911e11706a..ab9c202f0722f122a2e4d73230be7b166abad579 100644 (file)
--- a/servers/slapd/daemon.c
+++ b/servers/slapd/daemon.c
@@ -1560,8 +1560,9 @@ slapd_daemon_task(
                                        gid_t gid;
 
                                        if( getpeereid( s, &uid, &gid ) == 0 ) {
-                                               authid = ch_malloc( sizeof("uidNumber=+gidNumber=+,"
-                                                       "cn=peercred,cn=external,cn=auth") + 32);
+                                               authid = ch_malloc(
+                                                       sizeof("uidNumber=XXXXXX+gidNumber=XXXXXX,"
+                                                               "cn=peercred,cn=external,cn=auth"));
                                                sprintf(authid, "uidNumber=%d+gidNumber=%d,"
                                                        "cn=peercred,cn=external,cn=auth",
                                                        uid, gid);