The-Homebrew-Cloud/libfat
2
0
Fork 0
mirror of https://github.com/wiiu-env/libfat.git synced 2024-11-25 03:16:52 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix concurrency bugs in FAT_getAttr/FAT_setAttr. (#13)

Browse Source 
A relatively rare concurrency bug existed in these methods due to them not
correctly holding the partition lock during calls to
_FAT_directory_entryFromPath. That method can trigger reads to the partition.
Due to the lock not being held, another thread could still trigger a concurrent
read, potentially corrupting the cache. The fix is simply to ensure the lock is
held around the calls.
This commit is contained in:
Alex "Chadderz" Chadwick 2018-04-20 12:23:50 +01:00 committed by Dave Murphy
parent e7b04a73a0
commit a711f2bef2
1 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
source/fatfile.c
View File

@ -46,6 +46,7 @@
#include "lock.h"
bool _FAT_findEntry(const char *path, DIR_ENTRY *dirEntry) {
bool r;
PARTITION *partition = _FAT_partition_getPartitionFromPath(path);
// Check Partition
@ -61,8 +62,11 @@ bool _FAT_findEntry(const char *path, DIR_ENTRY *dirEntry) {
}
// Search for the file on the disc
return _FAT_directory_entryFromPath (partition, dirEntry, path, NULL);
_FAT_lock(&partition->lock);
r = _FAT_directory_entryFromPath (partition, dirEntry, path, NULL);
_FAT_unlock(&partition->lock);
return r;
}
int FAT_getAttr(const char *file) {
@ -92,16 +96,17 @@ int FAT_setAttr(const char *file, uint8_t attr) {
if (strchr (file, ':') != NULL)
return -1;
// Get DIR_ENTRY
if( !_FAT_directory_entryFromPath (partition, &dirEntry, file, NULL) )
return -1;
// Get Entry-End
entryEnd = dirEntry.dataEnd;
// Lock Partition
_FAT_lock(&partition->lock);
// Get DIR_ENTRY
if( !_FAT_directory_entryFromPath (partition, &dirEntry, file, NULL) ) {
_FAT_unlock(&partition->lock); // Unlock Partition
return -1;
}
// Get Entry-End
entryEnd = dirEntry.dataEnd;
// Write Data
_FAT_cache_writePartialSector (