Skip to content
This repository has been archived by the owner on Jul 10, 2022. It is now read-only.

LastAccessTime and FileChangeTime values are swapped. #2

Open
peteraritchie opened this issue Mar 26, 2022 · 0 comments · May be fixed by #3
Open

LastAccessTime and FileChangeTime values are swapped. #2

peteraritchie opened this issue Mar 26, 2022 · 0 comments · May be fixed by #3

Comments

@peteraritchie
Copy link

StandardInformation is not being instantiated properly at https://github.com/michaelkc/NtfsReader/blob/master/src/NtfsReader/System/IO/Filesystem/Ntfs/NtfsReader.cs#L1005:

                                    new StandardInformation(
                                        attributeStandardInformation->CreationTime,
                                        attributeStandardInformation->FileChangeTime,
                                        attributeStandardInformation->LastAccessTime
                                    );

where StandardInformation constructor is defined as:

            public StandardInformation(
                UInt64 creationTime,
                UInt64 lastAccessTime,
                UInt64 lastChangeTime
                )
peteraritchie added a commit to peteraritchie/NtfsReader that referenced this issue Mar 26, 2022
@peteraritchie
Fixed LastAccessTime and FileChangeTime being swapped
91e2e56 
The `StandardInformation` was not being initialized properly when processing attributes. `ProcessAttributes` was changed to instantiate `StandardInformation` passing the `LastAccessTime` and `FileChangeTime` attributes in the correct order.

Fixes michaelkc#2
@peteraritchie peteraritchie linked a pull request Mar 26, 2022 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed LastAccessTime and FileChangeTime being swapped peteraritchie/NtfsReader
1 participant
@peteraritchie