hardcoresushi/libcryfs
1
0
Fork 0
Code Issues Pull Requests Releases Activity

Fully support file access times

Browse Source
This commit is contained in:
Sebastian Messmer 2016-02-09 09:51:35 +01:00
parent bca4b6adf9
commit b0beb6b350
1 changed files with 14 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
src/filesystem/fsblobstore/utils/DirEntry.cpp
View File

@ -26,22 +26,19 @@ namespace cryfs {
*reinterpret_cast<mode_t*>(dest+offset) = mode;
offset += sizeof(mode_t);
//TODO Persist times, see comment in deserializeAndAddToVector()
//static_assert(sizeof(timespec) == 16, "Ensure platform independence of the serialization");
//*reinterpret_cast<timespec*>(dest+offset) = lastAccessTime;
//offset += sizeof(timespec);
//*reinterpret_cast<timespec*>(dest+offset) = lastModificationTime;
//offset += sizeof(timespec);
//*reinterpret_cast<timespec*>(dest+offset) = lastMetadataChangeTime;
//offset += sizeof(timespec);
static_assert(sizeof(timespec) == 16, "Ensure platform independence of the serialization");
*reinterpret_cast<timespec*>(dest+offset) = lastAccessTime;
offset += sizeof(timespec);
*reinterpret_cast<timespec*>(dest+offset) = lastModificationTime;
offset += sizeof(timespec);
*reinterpret_cast<timespec*>(dest+offset) = lastMetadataChangeTime;
offset += sizeof(timespec);
ASSERT(offset == serializedSize(), "Didn't write correct number of elements");
}
size_t DirEntry::serializedSize() const {
//TODO Persist times, see comment in deserializeAndAddToVector()
//return 1 + (name.size() + 1) + key.BINARY_LENGTH + sizeof(uid_t) + sizeof(gid_t) + sizeof(mode_t) + 3*sizeof(timespec);
return 1 + (name.size() + 1) + key.BINARY_LENGTH + sizeof(uid_t) + sizeof(gid_t) + sizeof(mode_t);
return 1 + (name.size() + 1) + key.BINARY_LENGTH + sizeof(uid_t) + sizeof(gid_t) + sizeof(mode_t) + 3*sizeof(timespec);
}
const char *DirEntry::deserializeAndAddToVector(const char *pos, vector<DirEntry> *result) {
@ -66,18 +63,12 @@ namespace cryfs {
mode_t mode = *(mode_t*)pos;
pos += sizeof(mode_t);
//TODO Persist times. This breaks compatibility though - so change cryfs::InnerConfig::HEADER
// This is already implemented, but I commented it out for now, because it would break compatibility.
//timespec lastAccessTime = *(timespec*)pos;
//pos += sizeof(timespec);
//timespec lastModificationTime = *(timespec*)pos;
//pos += sizeof(timespec);
//timespec lastMetadataChangeTime = *(timespec*)pos;
//pos += sizeof(timespec);
timespec lastAccessTime;
clock_gettime(CLOCK_REALTIME, &lastAccessTime);
timespec lastModificationTime = lastAccessTime;
timespec lastMetadataChangeTime = lastAccessTime;
timespec lastAccessTime = *(timespec*)pos;
pos += sizeof(timespec);
timespec lastModificationTime = *(timespec*)pos;
pos += sizeof(timespec);
timespec lastMetadataChangeTime = *(timespec*)pos;
pos += sizeof(timespec);
result->emplace_back(type, name, key, mode, uid, gid, lastAccessTime, lastModificationTime, lastMetadataChangeTime);
return pos;