ldap/servers/slapd/back-ldbm/dblayer.c | 73 +++++++++++-------------
ldap/servers/slapd/back-ldbm/dbsize.c | 7 +-
ldap/servers/slapd/back-ldbm/ldif2ldbm.c | 12 +--
ldap/servers/slapd/back-ldbm/misc.c | 8 +-
ldap/servers/slapd/config.c | 10 +--
ldap/servers/slapd/dse.c | 12 +--
ldap/servers/slapd/getfilelist.c | 4 -
ldap/servers/slapd/test-plugins/testdbinterop.c | 6 -
ldap/servers/slapd/tools/pwenc.c | 10 +--
9 files changed, 70 insertions(+), 72 deletions(-)
New commits:
commit 7d26ba1f6edbf1b0959c37fb638bc757cba9dfa6
Author: Rich Megginson <rmeggins(a)redhat.com>
Date: Tue Apr 2 17:55:14 2013 -0600
Ticket #574 - problems with dbcachesize disk space calculation
https://fedorahosted.org/389/ticket/574
Reviewed by: mreynolds (Thanks!)
Branch: master
Fix Description: 50% overhead for dbcachesize is too much - use 10%.
Based on empirical observations on RHEL 6.4 64-bit bdb 4.7.
There seems to be no way to ask bdb "Given a dbcachesize, what will be the
total size of all of the region files?".
Platforms tested: RHEL6 x86_64
Flag Day: no
Doc impact: no
diff --git a/ldap/servers/slapd/back-ldbm/dblayer.c
b/ldap/servers/slapd/back-ldbm/dblayer.c
index c9fecbe..0edd861 100644
--- a/ldap/servers/slapd/back-ldbm/dblayer.c
+++ b/ldap/servers/slapd/back-ldbm/dblayer.c
@@ -1397,7 +1397,8 @@ no_diskspace(struct ldbminfo *li, int dbenv_flags)
struct statvfs dbhome_buf;
struct statvfs db_buf;
int using_region_files = !(dbenv_flags & ( DB_PRIVATE | DB_SYSTEM_MEM));
- PRUint64 expected_siz = li->li_dbcachesize + li->li_dbcachesize/2; /* dbcache +
region files */
+ /* value of 10 == 10% == little more than the average overhead calculated for very
large files on 64-bit system for bdb 4.7 */
+ PRUint64 expected_siz = li->li_dbcachesize + li->li_dbcachesize/10; /* dbcache
+ region files */
PRUint64 fsiz;
char *region_dir;
commit 968c51168a575c0e61e1749fb62568ead9a87e8f
Author: Rich Megginson <rmeggins(a)redhat.com>
Date: Tue Apr 2 17:50:08 2013 -0600
Ticket #47312 - replace PR_GetFileInfo with PR_GetFileInfo64
https://fedorahosted.org/389/ticket/47312
Reviewed by: mreynolds (Thanks!)
Branch: master
Fix Description: The directory server deals with very large files. We need
to use PR_GetFileInfo64 to make sure we can accurately calculate the sizes of
these files. Note that many places didn't use the info.size field, but were
changed anyway in order to be consistent and discourage the use of
PR_GetFileInfo.
Platforms tested: RHEL6 x86_64
Flag Day: no
Doc impact: no
diff --git a/ldap/servers/slapd/back-ldbm/dblayer.c
b/ldap/servers/slapd/back-ldbm/dblayer.c
index c921526..c9fecbe 100644
--- a/ldap/servers/slapd/back-ldbm/dblayer.c
+++ b/ldap/servers/slapd/back-ldbm/dblayer.c
@@ -1086,7 +1086,7 @@ static int dblayer_grok_directory(char *directory, int flags)
char filename[MAXPATHLEN];
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
- PRFileInfo info;
+ PRFileInfo64 info;
dirhandle = PR_OpenDir(directory);
if (NULL == dirhandle)
@@ -1109,7 +1109,7 @@ static int dblayer_grok_directory(char *directory, int flags)
* With multiple instances of the backend the are now other directories
* in the db home directory. This function wasn't ment to deal with
* other directories, so we skip them. */
- if (PR_GetFileInfo(filename, &info) == PR_SUCCESS &&
+ if (PR_GetFileInfo64(filename, &info) == PR_SUCCESS &&
info.type == PR_FILE_DIRECTORY) {
/* go into it (instance dir) */
int retval = dblayer_grok_directory(filename, flags);
@@ -1360,7 +1360,7 @@ dblayer_get_full_inst_dir(struct ldbminfo *li, ldbm_instance *inst,
static PRUint64
dblayer_get_region_size(const char *dir)
{
- PRFileInfo info;
+ PRFileInfo64 info;
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
PRUint64 region_size = 0;
@@ -1377,7 +1377,7 @@ dblayer_get_region_size(const char *dir)
char filename[MAXPATHLEN];
PR_snprintf(filename, MAXPATHLEN, "%s/%s", dir,
direntry->name);
- if (PR_GetFileInfo(filename, &info) != PR_FAILURE){
+ if (PR_GetFileInfo64(filename, &info) != PR_FAILURE){
region_size += info.size;
}
}
@@ -1923,7 +1923,7 @@ dblayer_get_id2entry_size(ldbm_instance *inst)
{
struct ldbminfo *li = NULL;
char *id2entry_file = NULL;
- PRFileInfo info;
+ PRFileInfo64 info;
int rc;
char inst_dir[MAXPATHLEN], *inst_dirp;
@@ -1934,7 +1934,7 @@ dblayer_get_id2entry_size(ldbm_instance *inst)
inst_dirp = dblayer_get_full_inst_dir(li, inst, inst_dir, MAXPATHLEN);
id2entry_file = slapi_ch_smprintf("%s/%s", inst_dirp,
ID2ENTRY LDBM_FILENAME_SUFFIX);
- rc = PR_GetFileInfo(id2entry_file, &info);
+ rc = PR_GetFileInfo64(id2entry_file, &info);
slapi_ch_free_string(&id2entry_file);
if (rc) {
return 0;
@@ -2407,7 +2407,7 @@ dblayer_get_aux_id2entry_ext(backend *be, DB **ppDB, DB_ENV
**ppEnv,
int envflags = 0;
int dbflags = 0;
size_t cachesize;
- PRFileInfo prfinfo;
+ PRFileInfo64 prfinfo;
PRStatus prst;
char *id2entry_file = NULL;
char inst_dir[MAXPATHLEN];
@@ -2462,7 +2462,7 @@ dblayer_get_aux_id2entry_ext(backend *be, DB **ppDB, DB_ENV
**ppEnv,
}
priv->dblayer_log_directory = slapi_ch_strdup(priv->dblayer_home_directory);
- prst = PR_GetFileInfo(inst_dirp, &prfinfo);
+ prst = PR_GetFileInfo64(inst_dirp, &prfinfo);
if (PR_FAILURE == prst || PR_FILE_DIRECTORY != prfinfo.type)
{
LDAPDebug(LDAP_DEBUG_ANY,
@@ -2470,7 +2470,7 @@ dblayer_get_aux_id2entry_ext(backend *be, DB **ppDB, DB_ENV
**ppEnv,
goto done;
}
- prst = PR_GetFileInfo(priv->dblayer_home_directory, &prfinfo);
+ prst = PR_GetFileInfo64(priv->dblayer_home_directory, &prfinfo);
if (PR_SUCCESS == prst)
{
ldbm_delete_dirs(priv->dblayer_home_directory);
@@ -4913,7 +4913,7 @@ static int read_metadata(struct ldbminfo *li)
char *nextline;
char **dirp;
PRFileDesc *prfd;
- PRFileInfo prfinfo;
+ PRFileInfo64 prfinfo;
int return_value = 0;
PRInt32 byte_count = 0;
char attribute[512];
@@ -4931,7 +4931,7 @@ static int read_metadata(struct ldbminfo *li)
PR_snprintf(filename,sizeof(filename),"%s/guardian",priv->dblayer_home_directory);
memset(&prfinfo, '\0', sizeof(PRFileInfo));
- (void)PR_GetFileInfo(filename, &prfinfo);
+ (void)PR_GetFileInfo64(filename, &prfinfo);
prfd = PR_Open(filename,PR_RDONLY,priv->dblayer_file_mode);
if (NULL == prfd || 0 == prfinfo.size) {
@@ -5263,7 +5263,7 @@ dblayer_delete_database_ex(struct ldbminfo *li, char *instance, char
*cldir)
Object *inst_obj;
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
- PRFileInfo fileinfo;
+ PRFileInfo64 fileinfo;
char filename[MAXPATHLEN];
char *log_dir;
int ret;
@@ -5331,7 +5331,7 @@ dblayer_delete_database_ex(struct ldbminfo *li, char *instance, char
*cldir)
/* Do not call PR_Delete on the instance directories if they exist.
* It would not work, but we still should not do it. */
- rval_tmp = PR_GetFileInfo(filename, &fileinfo);
+ rval_tmp = PR_GetFileInfo64(filename, &fileinfo);
if (rval_tmp == PR_SUCCESS && fileinfo.type != PR_FILE_DIRECTORY)
{
/* Skip deleting log files; that should be handled below.
@@ -5381,28 +5381,24 @@ int dblayer_delete_database(struct ldbminfo *li)
/*
* Return the size of the database (in kilobytes). XXXggood returning
* the size in units of kb is really a hack, and is done because we
- * don't have NSPR support for 64-bit file offsets.
+ * didn't have NSPR support for 64-bit file offsets originally (now we do)
* Caveats:
* - We can still return incorrect results if an individual file is
* larger than fit in a PRUint32.
- * - PR_GetFileInfo doesn't do any special processing for symlinks,
+ * - PR_GetFileInfo64 doesn't do any special processing for symlinks,
* nor does it inform us if the file is a symlink. Nice. So if
* a file in the db directory is a symlink, the size we return
* will probably be way too small.
*/
int dblayer_database_size(struct ldbminfo *li, unsigned int *size)
{
- dblayer_private *priv = NULL;
+ dblayer_private *priv = NULL;
int return_value = 0;
char filename[MAXPATHLEN];
PRDir *dirhandle = NULL;
- /*
- * XXXggood - NSPR will only give us an unsigned 32-bit quantity for
- * file sizes. This is bad. Files can be bigger than that these days.
- */
unsigned int cumulative_size = 0;
unsigned int remainder = 0;
- PRFileInfo info;
+ PRFileInfo64 info;
PR_ASSERT(NULL != li);
priv = (dblayer_private*)li->li_dblayer_private;
@@ -5418,7 +5414,7 @@ int dblayer_database_size(struct ldbminfo *li, unsigned int *size)
break;
}
PR_snprintf(filename,MAXPATHLEN,
"%s/%s",priv->dblayer_home_directory,direntry->name);
- return_value = PR_GetFileInfo(filename, &info);
+ return_value = PR_GetFileInfo64(filename, &info);
if (PR_SUCCESS == return_value)
{
cumulative_size += (info.size / 1024);
@@ -5459,14 +5455,14 @@ static int count_dbfiles_in_dir(char *directory, int *count, int
recurse)
if (NULL != dirhandle) {
PRDirEntry *direntry = NULL;
char *direntry_name;
- PRFileInfo info;
+ PRFileInfo64 info;
while (NULL != (direntry = PR_ReadDir(dirhandle, PR_SKIP_DOT | PR_SKIP_DOT_DOT)))
{
if (NULL == direntry->name) {
break;
}
direntry_name = PR_smprintf("%s/%s", directory,
direntry->name);
- if ((PR_GetFileInfo(direntry_name, &info) == PR_SUCCESS) &&
+ if ((PR_GetFileInfo64(direntry_name, &info) == PR_SUCCESS) &&
(PR_FILE_DIRECTORY == info.type) && recurse) {
/* Recurse into this directory but not any further. This is
* because each instance gets its own directory, but in those
@@ -5736,7 +5732,7 @@ dblayer_copy_directory(struct ldbminfo *li,
if (NULL == new_dest_dir) {
/* Need to create the new directory where the files will be
* copied to. */
- PRFileInfo info;
+ PRFileInfo64 info;
char *prefix = "";
char mysep = 0;
@@ -5757,7 +5753,7 @@ dblayer_copy_directory(struct ldbminfo *li,
new_dest_dir = slapi_ch_smprintf("%s/%s",
dest_dir, relative_instance_name);
/* } */
- if (PR_SUCCESS == PR_GetFileInfo(new_dest_dir, &info))
+ if (PR_SUCCESS == PR_GetFileInfo64(new_dest_dir, &info))
{
ldbm_delete_dirs(new_dest_dir);
}
@@ -6251,7 +6247,7 @@ int dblayer_delete_transaction_logs(const char * log_dir)
PRDirEntry *direntry = NULL;
int is_a_logfile = 0;
int pre=0;
- PRFileInfo info ;
+ PRFileInfo64 info ;
while (NULL != (direntry =
PR_ReadDir(dirhandle, PR_SKIP_DOT | PR_SKIP_DOT_DOT)))
@@ -6263,7 +6259,7 @@ int dblayer_delete_transaction_logs(const char * log_dir)
break;
}
PR_snprintf(filename1, MAXPATHLEN, "%s/%s", log_dir,
direntry->name);
- pre = PR_GetFileInfo(filename1, &info);
+ pre = PR_GetFileInfo64(filename1, &info);
if (pre == PR_SUCCESS && PR_FILE_DIRECTORY == info.type) {
continue;
}
@@ -6315,7 +6311,7 @@ static int dblayer_copy_dirand_contents(char* src_dir, char*
dst_dir, int mode,
char filename2[MAXPATHLEN];
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
- PRFileInfo info;
+ PRFileInfo64 info;
dirhandle = PR_OpenDir(src_dir);
if (NULL != dirhandle)
@@ -6334,7 +6330,7 @@ static int dblayer_copy_dirand_contents(char* src_dir, char*
dst_dir, int mode,
LDAPDebug(LDAP_DEBUG_ANY, "Moving file %s\n",
filename2, 0, 0);
/* Is this entry a directory? */
- tmp_rval = PR_GetFileInfo(filename1, &info);
+ tmp_rval = PR_GetFileInfo64(filename1, &info);
if (tmp_rval == PR_SUCCESS && PR_FILE_DIRECTORY == info.type)
{
PR_MkDir(filename2,NEWDIR_MODE);
@@ -6375,7 +6371,7 @@ static int dblayer_fri_trim(char *fri_dir_path, char* bename)
char filename[MAXPATHLEN];
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
- PRFileInfo info;
+ PRFileInfo64 info;
dirhandle = PR_OpenDir(fri_dir_path);
if (NULL != dirhandle)
@@ -6391,7 +6387,7 @@ static int dblayer_fri_trim(char *fri_dir_path, char* bename)
PR_snprintf(filename, MAXPATHLEN, "%s/%s", fri_dir_path,
direntry->name);
/* Is this entry a directory? */
- tmp_rval = PR_GetFileInfo(filename, &info);
+ tmp_rval = PR_GetFileInfo64(filename, &info);
if (tmp_rval == PR_SUCCESS && PR_FILE_DIRECTORY == info.type)
{
if(strcmp(direntry->name,bename)!=0)
@@ -6504,7 +6500,7 @@ int dblayer_restore(struct ldbminfo *li, char *src_dir, Slapi_Task
*task, char *
char filename2[MAXPATHLEN];
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
- PRFileInfo info;
+ PRFileInfo64 info;
ldbm_instance *inst = NULL;
int seen_logfiles = 0; /* Tells us if we restored any logfiles */
int is_a_logfile = 0;
@@ -6592,7 +6588,7 @@ int dblayer_restore(struct ldbminfo *li, char *src_dir, Slapi_Task
*task, char *
src_dir, direntry->name);
if(!frirestore || strcmp(direntry->name,bename)==0)
{
- tmp_rval = PR_GetFileInfo(filename1, &info);
+ tmp_rval = PR_GetFileInfo64(filename1, &info);
if (tmp_rval == PR_SUCCESS && PR_FILE_DIRECTORY == info.type) {
/* Is it CHANGELOG_BACKUPDIR? */
if (0 == strcmp(CHANGELOG_BACKUPDIR, direntry->name)) {
@@ -6687,7 +6683,7 @@ int dblayer_restore(struct ldbminfo *li, char *src_dir, Slapi_Task
*task, char *
/* Is this entry a directory? */
PR_snprintf(filename1, sizeof(filename1), "%s/%s",
real_src_dir, direntry->name);
- tmp_rval = PR_GetFileInfo(filename1, &info);
+ tmp_rval = PR_GetFileInfo64(filename1, &info);
if (tmp_rval == PR_SUCCESS && PR_FILE_DIRECTORY == info.type) {
/* This is an instance directory. It contains the *.db#
* files for the backend instance.
@@ -7045,10 +7041,10 @@ int dblayer_update_db_ext(ldbm_instance *inst, char *oldext, char
*newext)
NULL != a;
a = (struct attrinfo *)avl_getnext())
{
- PRFileInfo info;
+ PRFileInfo64 info;
ofile = slapi_ch_smprintf("%s/%s%s", inst_dirp, a->ai_type,
oldext);
- if (PR_GetFileInfo(ofile, &info) != PR_SUCCESS)
+ if (PR_GetFileInfo64(ofile, &info) != PR_SUCCESS)
{
slapi_ch_free_string(&ofile);
continue;
diff --git a/ldap/servers/slapd/back-ldbm/dbsize.c
b/ldap/servers/slapd/back-ldbm/dbsize.c
index 43defe1..251ac03 100644
--- a/ldap/servers/slapd/back-ldbm/dbsize.c
+++ b/ldap/servers/slapd/back-ldbm/dbsize.c
@@ -47,16 +47,17 @@
#include "back-ldbm.h"
+/* TODO: make this a 64-bit return value */
int
ldbm_db_size( Slapi_PBlock *pb )
{
struct ldbminfo *li;
- unsigned int size;
+ unsigned int size; /* TODO: make this a 64-bit return value */
int rc;
slapi_pblock_get( pb, SLAPI_PLUGIN_PRIVATE, &li );
- rc = dblayer_database_size(li, &size);
- slapi_pblock_set( pb, SLAPI_DBSIZE, &size );
+ rc = dblayer_database_size(li, &size); /* TODO: make this a 64-bit return value */
+ slapi_pblock_set( pb, SLAPI_DBSIZE, &size ); /* TODO: make this a 64-bit return
value */
return rc;
}
diff --git a/ldap/servers/slapd/back-ldbm/ldif2ldbm.c
b/ldap/servers/slapd/back-ldbm/ldif2ldbm.c
index 8ef1d57..8818ba7 100644
--- a/ldap/servers/slapd/back-ldbm/ldif2ldbm.c
+++ b/ldap/servers/slapd/back-ldbm/ldif2ldbm.c
@@ -2620,7 +2620,7 @@ int ldbm_back_upgradedb(Slapi_PBlock *pb)
char inst_dir[MAXPATHLEN];
char *inst_dirp = NULL;
int cnt = 0;
- PRFileInfo info = {0};
+ PRFileInfo64 info = {0};
PRUint32 dbversion_flags = DBVERSION_ALL;
slapi_pblock_get(pb, SLAPI_SEQ_TYPE, &up_flags);
@@ -2750,7 +2750,7 @@ int ldbm_back_upgradedb(Slapi_PBlock *pb)
orig_dest_dir = dest_dir;
normalize_dir(dest_dir);
/* clean up the backup dir first, then create it */
- rval = PR_GetFileInfo(dest_dir, &info);
+ rval = PR_GetFileInfo64(dest_dir, &info);
if (PR_SUCCESS == rval)
{
if (PR_FILE_DIRECTORY == info.type) /* directory exists */
@@ -3071,7 +3071,7 @@ int upgradedb_delete_indices_4cmd(ldbm_instance *inst, int flags)
while (NULL != (direntry =
PR_ReadDir(dirhandle, PR_SKIP_DOT | PR_SKIP_DOT_DOT)))
{
- PRFileInfo info;
+ PRFileInfo64 info;
int len;
if (! direntry->name)
@@ -3086,7 +3086,7 @@ int upgradedb_delete_indices_4cmd(ldbm_instance *inst, int flags)
fullpathp = (char *)slapi_ch_malloc(len);
}
sprintf(fullpathp, "%s/%s", inst_dirp, direntry->name);
- rval = PR_GetFileInfo(fullpathp, &info);
+ rval = PR_GetFileInfo64(fullpathp, &info);
if (PR_SUCCESS == rval && PR_FILE_DIRECTORY != info.type)
{
PR_Delete(fullpathp);
@@ -3530,7 +3530,7 @@ int ldbm_back_upgradednformat(Slapi_PBlock *pb)
char *instance_name = NULL;
backend *be = NULL;
PRStatus prst = 0;
- PRFileInfo prfinfo = {0};
+ PRFileInfo64 prfinfo = {0};
PRDir *dirhandle = NULL;
PRDirEntry *direntry = NULL;
size_t id2entrylen = 0;
@@ -3579,7 +3579,7 @@ int ldbm_back_upgradednformat(Slapi_PBlock *pb)
slapi_pblock_get(pb, SLAPI_SEQ_VAL, &rawworkdbdir);
normalize_dir(rawworkdbdir); /* remove trailing spaces and slashes */
- prst = PR_GetFileInfo(rawworkdbdir, &prfinfo);
+ prst = PR_GetFileInfo64(rawworkdbdir, &prfinfo);
if (PR_FAILURE == prst || PR_FILE_DIRECTORY != prfinfo.type) {
slapi_log_error(SLAPI_LOG_FATAL, "Upgrade DN Format",
"Working DB instance dir %s is not a directory\n",
diff --git a/ldap/servers/slapd/back-ldbm/misc.c b/ldap/servers/slapd/back-ldbm/misc.c
index 56a8298..87262c5 100644
--- a/ldap/servers/slapd/back-ldbm/misc.c
+++ b/ldap/servers/slapd/back-ldbm/misc.c
@@ -296,7 +296,7 @@ ldbm_delete_dirs(char *path)
PRDirEntry *direntry = NULL;
char fullpath[MAXPATHLEN];
int rval = 0;
- PRFileInfo info;
+ PRFileInfo64 info;
dirhandle = PR_OpenDir(path);
if (! dirhandle)
@@ -312,7 +312,7 @@ ldbm_delete_dirs(char *path)
break;
PR_snprintf(fullpath, MAXPATHLEN, "%s/%s", path, direntry->name);
- rval = PR_GetFileInfo(fullpath, &info);
+ rval = PR_GetFileInfo64(fullpath, &info);
if (PR_SUCCESS == rval)
{
if (PR_FILE_DIRECTORY == info.type)
@@ -343,11 +343,11 @@ get_sep(char *path)
int
mkdir_p(char *dir, unsigned int mode)
{
- PRFileInfo info;
+ PRFileInfo64 info;
int rval;
char sep = get_sep(dir);
- rval = PR_GetFileInfo(dir, &info);
+ rval = PR_GetFileInfo64(dir, &info);
if (PR_SUCCESS == rval)
{
if (PR_FILE_DIRECTORY != info.type) /* not a directory */
diff --git a/ldap/servers/slapd/config.c b/ldap/servers/slapd/config.c
index 3edc24b..724b8d0 100644
--- a/ldap/servers/slapd/config.c
+++ b/ldap/servers/slapd/config.c
@@ -157,10 +157,10 @@ int
slapd_bootstrap_config(const char *configdir)
{
char configfile[MAXPATHLEN+1];
- PRFileInfo prfinfo;
- int rc = 0; /* Fail */
+ PRFileInfo64 prfinfo;
+ int rc = 0; /* Fail */
int done = 0;
- PRInt32 nr = 0;
+ PRInt32 nr = 0;
PRFileDesc *prfd = 0;
char *buf = 0;
char *lastp = 0;
@@ -182,7 +182,7 @@ slapd_bootstrap_config(const char *configdir)
rc = dse_check_file(configfile, tmpfile);
}
- if ( (rc = PR_GetFileInfo( configfile, &prfinfo )) != PR_SUCCESS )
+ if ( (rc = PR_GetFileInfo64( configfile, &prfinfo )) != PR_SUCCESS )
{
PRErrorCode prerr = PR_GetError();
slapi_log_error(SLAPI_LOG_FATAL, "config", "The given config file %s
could not be accessed, " SLAPI_COMPONENT_NAME_NSPR " error %d (%s)\n",
@@ -203,7 +203,7 @@ slapd_bootstrap_config(const char *configdir)
buf = slapi_ch_malloc( prfinfo.size + 1 );
if (( nr = slapi_read_buffer( prfd, buf, prfinfo.size )) < 0 )
{
- slapi_log_error(SLAPI_LOG_FATAL, "config", "Could only read %d of %d
bytes from config file %s\n",
+ slapi_log_error(SLAPI_LOG_FATAL, "config", "Could only read %d of %ld
bytes from config file %s\n",
nr, prfinfo.size, configfile);
rc = 0; /* Fail */
done= 1;
diff --git a/ldap/servers/slapd/dse.c b/ldap/servers/slapd/dse.c
index 7896fc9..ab7448c 100644
--- a/ldap/servers/slapd/dse.c
+++ b/ldap/servers/slapd/dse.c
@@ -660,9 +660,9 @@ int
dse_check_file(char *filename, char *backupname)
{
int rc= 0; /* Fail */
- PRFileInfo prfinfo;
+ PRFileInfo64 prfinfo;
- if (PR_GetFileInfo( filename, &prfinfo ) == PR_SUCCESS) {
+ if (PR_GetFileInfo64( filename, &prfinfo ) == PR_SUCCESS) {
if ( prfinfo.size > 0)
return (1);
else {
@@ -675,7 +675,7 @@ dse_check_file(char *filename, char *backupname)
else
return (0);
- if ( PR_GetFileInfo( filename, &prfinfo ) == PR_SUCCESS && prfinfo.size
> 0 ) {
+ if ( PR_GetFileInfo64( filename, &prfinfo ) == PR_SUCCESS && prfinfo.size
> 0 ) {
slapi_log_error(SLAPI_LOG_FATAL, "dse",
"The configuration file %s was restored from backup %s\n",
filename, backupname);
return (1);
@@ -696,7 +696,7 @@ dse_read_one_file(struct dse *pdse, const char *filename, Slapi_PBlock
*pb,
char *lastp = NULL;
int rc= 0; /* Fail */
PRInt32 nr = 0;
- PRFileInfo prfinfo;
+ PRFileInfo64 prfinfo;
PRFileDesc *prfd = 0;
int schema_flags = 0;
@@ -709,7 +709,7 @@ dse_read_one_file(struct dse *pdse, const char *filename, Slapi_PBlock
*pb,
if (!rc)
rc = dse_check_file((char *)filename, pdse->dse_fileback);
- if ( (rc = PR_GetFileInfo( filename, &prfinfo )) != PR_SUCCESS )
+ if ( (rc = PR_GetFileInfo64( filename, &prfinfo )) != PR_SUCCESS )
{
slapi_log_error(SLAPI_LOG_FATAL, "dse",
"The configuration file %s could not be accessed, error
%d\n",
@@ -733,7 +733,7 @@ dse_read_one_file(struct dse *pdse, const char *filename, Slapi_PBlock
*pb,
if (( nr = slapi_read_buffer( prfd, buf, prfinfo.size )) < 0 )
{
slapi_log_error(SLAPI_LOG_FATAL, "dse",
- "Could only read %d of %d bytes from config file
%s\n",
+ "Could only read %d of %ld bytes from config file
%s\n",
nr, prfinfo.size, filename);
rc = 0; /* Fail */
done= 1;
diff --git a/ldap/servers/slapd/getfilelist.c b/ldap/servers/slapd/getfilelist.c
index c8dd8f9..64c3e51 100644
--- a/ldap/servers/slapd/getfilelist.c
+++ b/ldap/servers/slapd/getfilelist.c
@@ -93,10 +93,10 @@ free_string(caddr_t data)
static int
file_is_type_x(const char *dirname, const char *filename, PRFileType x)
{
- struct PRFileInfo inf;
+ struct PRFileInfo64 inf;
int status = 0;
char *fullpath = slapi_ch_smprintf("%s/%s", dirname, filename);
- if (PR_SUCCESS == PR_GetFileInfo(fullpath, &inf) &&
+ if (PR_SUCCESS == PR_GetFileInfo64(fullpath, &inf) &&
inf.type == x)
status = 1;
diff --git a/ldap/servers/slapd/test-plugins/testdbinterop.c
b/ldap/servers/slapd/test-plugins/testdbinterop.c
index 0397654..5ff730f 100644
--- a/ldap/servers/slapd/test-plugins/testdbinterop.c
+++ b/ldap/servers/slapd/test-plugins/testdbinterop.c
@@ -155,7 +155,7 @@ db_put_dn(char *data_dn)
int ret;
char *db_path = DATABASE;
char *db_path_bak = DATABASE_BACK;
- PRFileInfo info;
+ PRFileInfo64 info;
PRFileDesc *prfd;
PRInt32 data_sz;
char *data_dnp = NULL;
@@ -165,10 +165,10 @@ db_put_dn(char *data_dn)
}
PR_Lock(db_lock);
/* if db_path is a directory, rename it */
- ret = PR_GetFileInfo(db_path, &info);
+ ret = PR_GetFileInfo64(db_path, &info);
if (PR_SUCCESS == ret) {
if (PR_FILE_DIRECTORY == info.type) { /* directory */
- ret = PR_GetFileInfo(db_path_bak, &info);
+ ret = PR_GetFileInfo64(db_path_bak, &info);
if (PR_SUCCESS == ret) {
if (PR_FILE_DIRECTORY != info.type) { /* not a directory */
PR_Delete(db_path_bak);
diff --git a/ldap/servers/slapd/tools/pwenc.c b/ldap/servers/slapd/tools/pwenc.c
index 221398c..29f3e1a 100644
--- a/ldap/servers/slapd/tools/pwenc.c
+++ b/ldap/servers/slapd/tools/pwenc.c
@@ -304,10 +304,10 @@ static int
slapd_config(const char *configdir, const char *givenconfigfile)
{
char configfile[MAXPATHLEN+1];
- PRFileInfo prfinfo;
- int rc = 0; /* Fail */
+ PRFileInfo64 prfinfo;
+ int rc = 0; /* Fail */
int done = 0;
- PRInt32 nr = 0;
+ PRInt32 nr = 0;
PRFileDesc *prfd = 0;
char *buf = 0;
char *lastp = 0;
@@ -318,7 +318,7 @@ slapd_config(const char *configdir, const char *givenconfigfile)
}
PR_snprintf(configfile, sizeof(configfile), "%s/%s", configdir,
givenconfigfile);
- if ( (rc = PR_GetFileInfo( configfile, &prfinfo )) != PR_SUCCESS )
+ if ( (rc = PR_GetFileInfo64( configfile, &prfinfo )) != PR_SUCCESS )
{
fprintf(stderr,
"The given config file %s could not be accessed, error %d\n",
@@ -340,7 +340,7 @@ slapd_config(const char *configdir, const char *givenconfigfile)
if (( nr = slapi_read_buffer( prfd, buf, prfinfo.size )) < 0 )
{
fprintf(stderr,
- "Could only read %d of %d bytes from config file %s\n",
+ "Could only read %d of %ld bytes from config file %s\n",
nr, prfinfo.size, configfile);
exit( 1 );
}