safer defaults ...

[openldap] / servers / slurpd / lock.c

diff --git a/servers/slurpd/lock.c b/servers/slurpd/lock.c
index d457cdfc21ae62aa655692e0bef074e06d05c21f..8a8fa65f98c447193addb3ad8e5a61dd6883ce52 100644 (file)
--- a/servers/slurpd/lock.c
+++ b/servers/slurpd/lock.c
@@ -1,6 +1,6 @@
 /* $OpenLDAP$ */
 /*
- * Copyright 1998-2002 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2003 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  */
 /*
@@ -47,12 +47,12 @@ lock_fopen(
        char    buf[MAXPATHLEN];
 
        /* open the lock file */
-       strcpy( buf, fname );
-       strcat( buf, ".lock" );
+       snprintf( buf, sizeof buf, "%s.lock", fname );
+
        if ( (*lfp = fopen( buf, "w" )) == NULL ) {
 #ifdef NEW_LOGGING
-               LDAP_LOG (( "lock", LDAP_LEVEL_ERR, "lock_fopen: "
-                       "Error: could not open \"%s\"\n", buf ));
+               LDAP_LOG ( SLURPD, ERR, "lock_fopen: "
+                       "Error: could not open \"%s\"\n", buf, 0, 0 );
 #else
                Debug( LDAP_DEBUG_ANY,
                        "Error: could not open \"%s\"\n", buf, 0, 0 );
@@ -66,8 +66,8 @@ lock_fopen(
        /* open the log file */
        if ( (fp = fopen( fname, type )) == NULL ) {
 #ifdef NEW_LOGGING
-               LDAP_LOG (( "lock", LDAP_LEVEL_ERR, "lock_fopen: "
-                       "Error: could not open \"%s\"\n", fname ));
+               LDAP_LOG ( SLURPD, ERR, "lock_fopen: "
+                       "Error: could not open \"%s\"\n", fname, 0, 0 );
 #else
                Debug( LDAP_DEBUG_ANY,
                        "Error: could not open \"%s\"\n", fname, 0, 0 );
@@ -110,9 +110,9 @@ acquire_lock(
 {
     if (( *rfp = lock_fopen( file, "r+", lfp )) == NULL ) {
 #ifdef NEW_LOGGING
-       LDAP_LOG (( "lock", LDAP_LEVEL_ERR, "acquire_lock: "
+       LDAP_LOG ( SLURPD, ERR, "acquire_lock: "
                "Error: acquire_lock(%ld): Could not acquire lock on \"%s\"\n",
-               (long) getpid(), file ));
+               (long) getpid(), file, 0 );
 #else
        Debug( LDAP_DEBUG_ANY,
                "Error: acquire_lock(%ld): Could not acquire lock on \"%s\"\n",
@@ -138,9 +138,9 @@ relinquish_lock(
 {
     if ( lock_fclose( rfp, lfp ) == EOF ) {
 #ifdef NEW_LOGGING
-       LDAP_LOG (( "lock", LDAP_LEVEL_ERR, "relinguish_lock: "
+       LDAP_LOG ( SLURPD, ERR, "relinguish_lock: "
                "Error: relinquish_lock (%ld): Error closing \"%s\"\n",
-               (long) getpid(), file ));
+               (long) getpid(), file, 0 );
 #else
        Debug( LDAP_DEBUG_ANY,
                "Error: relinquish_lock (%ld): Error closing \"%s\"\n",