diff options
author | Xin Li <delphij@google.com> | 2021-11-04 17:32:26 -0700 |
---|---|---|
committer | Xin Li <delphij@google.com> | 2021-11-04 17:32:26 -0700 |
commit | d46acd8389628eedce39706c71e71f269008a0b4 (patch) | |
tree | 8259c9f10a50a8b1a155815a57956091c250bde3 | |
parent | 62a280b2c2489b2fa99a54e806d987bd0bf0039a (diff) | |
download | fsck_msdos-d46acd8389628eedce39706c71e71f269008a0b4.tar.gz |
Import revision 890cae197737b463e56d1cc5a3f61f84cb49c807 from FreeBSD.upstream-master
Change-Id: Ibf4413e89d25b9b3bb365c9ac4de7ce6bf99e53e
-rw-r--r-- | dir.c | 21 |
1 files changed, 17 insertions, 4 deletions
@@ -400,8 +400,21 @@ checksize(struct fat_descriptor *fat, u_char *p, struct dosDirEntry *dir) if (dir->head == CLUST_FREE) { physicalSize = 0; } else { - if (!fat_is_valid_cl(fat, dir->head)) - return FSERROR; + if (!fat_is_valid_cl(fat, dir->head) || !fat_is_cl_head(fat, dir->head)) { + pwarn("Directory entry %s of size %u referencing invalid cluster %u\n", + fullpath(dir), dir->size, dir->head); + if (ask(1, "Truncate")) { + p[28] = p[29] = p[30] = p[31] = 0; + p[26] = p[27] = 0; + if (boot->ClustMask == CLUST32_MASK) + p[20] = p[21] = 0; + dir->size = 0; + dir->head = CLUST_FREE; + return FSDIRMOD; + } else { + return FSERROR; + } + } ret = checkchain(fat, dir->head, &chainsize); /* * Upon return, chainsize would hold the chain length @@ -422,8 +435,8 @@ checksize(struct fat_descriptor *fat, u_char *p, struct dosDirEntry *dir) physicalSize = (u_int64_t)chainsize * boot->ClusterSize; } if (physicalSize < dir->size) { - pwarn("size of %s is %u, should at most be %" PRIu64 "\n", - fullpath(dir), dir->size, physicalSize); + pwarn("size of %s is %u, should at most be %ju\n", + fullpath(dir), dir->size, (uintmax_t)physicalSize); if (ask(1, "Truncate")) { dir->size = physicalSize; p[28] = (u_char)physicalSize; |