Bug 29403 - st_atim, st_mtim, st_ctim stat struct members are missing on microblaze with largefile
Summary: st_atim, st_mtim, st_ctim stat struct members are missing on microblaze with ...
Status: RESOLVED FIXED
Alias: None
Product: glibc
Classification: Unclassified
Component: libc (show other bugs)
Version: 2.36
: P2 normal
Target Milestone: 2.36
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-25 09:31 UTC by Arnout Vandecappelle
Modified: 2022-08-02 15:42 UTC (History)
3 users (show)

See Also:
Host:
Target:
Build:
Last reconfirmed: 2022-07-25 00:00:00
fweimer: security-

Attachments
Make struct stat posix conformant on microblaze with __USE_FILE_OFFSET64 (793 bytes, patch)
2022-07-25 09:31 UTC, Arnout Vandecappelle 		Details | Diff
Make struct stat posix conformant on microblaze with __USE_FILE_OFFSET64 (797 bytes, patch)
2022-07-25 09:56 UTC, Arnout Vandecappelle 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Arnout Vandecappelle 2022-07-25 09:31:39 UTC 
Created attachment 14230 [details]
Make struct stat posix conformant on microblaze with __USE_FILE_OFFSET64

Commit a06b40cdf5ba0d2ab4f9b4c77d21e45ff284fac7 updated stat.h to use
__USE_XOPEN2K8 instead of __USE_MISC to add the st_atim, st_mtim and
st_ctim members to struct stat. However, for microblaze, there are two
definitions of struct stat, depending on the __USE_FILE_OFFSET64 macro.
The second one was not updated.

Fix attached.
Comment 1 Arnout Vandecappelle 2022-07-25 09:56:30 UTC 
Created attachment 14231 [details]
Make struct stat posix conformant on microblaze with __USE_FILE_OFFSET64

Original patch was based on an old branch - rebased it on master.
Comment 2 Adhemerval Zanella 2022-07-25 14:05:46 UTC 
Thanks for cathing it, I will install it.
Comment 3 Adhemerval Zanella 2022-07-25 14:08:21 UTC 
Fixed on 2.36.