Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix use of statvfs on 32-bit platforms with large file support #296

Closed
Julien-Elie opened this issue Feb 11, 2024 · 0 comments
Closed

Fix use of statvfs on 32-bit platforms with large file support #296

Julien-Elie opened this issue Feb 11, 2024 · 0 comments
Assignees
@Julien-Elie
Labels
bug Something isn't working C: general Related to several components P: medium Medium priority
Milestone
2.7.2

Comments

@Julien-Elie
Copy link
Contributor

A long type (4 bytes) cannot contain the available number of inodes and like on 32-bit architectures with large file support enabled. A long long type is needed (8 bytes).
It affects inndf, contrib/expirectl and a cleaning function in timecaf.
@Julien-Elie Julien-Elie added bug Something isn't working C: general Related to several components P: medium Medium priority labels Feb 11, 2024
@Julien-Elie Julien-Elie added this to the 2.7.2 milestone Feb 11, 2024
@Julien-Elie Julien-Elie self-assigned this Feb 11, 2024
Julien-Elie added a commit that referenced this issue Feb 11, 2024
@Julien-Elie
Fix use of statvfs on 32-bit platforms
b69239c 
A long type (4 bytes) cannot contain the available number of inodes and
like on 32-bit architectures with large file support enabled.  A long
long type is needed (8 bytes).

see #296
Julien-Elie added a commit that referenced this issue Feb 11, 2024
@Julien-Elie
inndf: Use long long for 64-bit numbers (on i686)
1a16a39 
The number was stored in a long int, which is not enough to store
64-bit numbers on 32-bit platforms.  We now store it in a long long
int.

inndf was otherwise returning 0 because of an overflow, even when
compiled with large file support.

close #296
Julien-Elie added a commit that referenced this issue Feb 11, 2024
@Julien-Elie
Fix use of statvfs on 32-bit platforms
2567813 
A long type (4 bytes) cannot contain the available number of inodes and
like on 32-bit architectures with large file support enabled.  A long
long type is needed (8 bytes).

see #296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Julien-Elie Julien-Elie

Labels
bug Something isn't working C: general Related to several components P: medium Medium priority
Milestone
2.7.2
Development

No branches or pull requests
1 participant
@Julien-Elie