#47312 replace PR_GetFileInfo with PR_GetFileInfo64
Closed: wontfix None Opened 11 years ago by rmeggins.

This is with the latest code, on a RHEL 6.4 64-bit machine.

[02/Apr/2013:17:26:38 -0400] - 389-Directory/1.3.1.pre.a1.git0472b98 B2013.092.2034 starting up
[02/Apr/2013:17:26:38 -0400] - WARNING: two: entry cache size 3897032704B is less than db size 2736742400B; We recommend to increase the entry cache size nsslapd-cachememsize.
[02/Apr/2013:17:26:38 -0400] - WARNING: one: entry cache size 3897032704B is less than db size 2736742400B; We recommend to increase the entry cache size nsslapd-cachememsize.

3897032704B is greater than 2736742400B


Looks like the problem is here:

{{{
size_t
dblayer_get_id2entry_size(ldbm_instance *inst)
...
PRFileInfo info;
...
rc = PR_GetFileInfo(id2entry_file, &info);
}}}

should use PRFileInfo64 instead.

There are several places in the code where we should use PR_GetFileInfo64 instead of PR_GetFileInfo. We need to be able to handle these very large files.

commit 968c511
Author: Rich Megginson rmeggins@redhat.com
Date: Tue Apr 2 17:50:08 2013 -0600

0001-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.2.patch
0001-Ticket-47312-replace-PR_GetFileInfo-with-PR_GetFileI.2.patch

commit 9d5dedd
Author: Rich Megginson rmeggins@redhat.com
Date: Wed Apr 3 16:19:30 2013 -0600

Metadata Update from @rmeggins:
- Issue assigned to rmeggins
- Issue set to the milestone: 1.3.1

7 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/649

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)

3 years ago

Login to comment on this ticket.

Metadata