[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH v2 10/13] vvfat: correctly generate numeric-tail of
From: |
Hervé Poussineau |
Subject: |
[Qemu-block] [PATCH v2 10/13] vvfat: correctly generate numeric-tail of short file names |
Date: |
Mon, 22 May 2017 23:12:02 +0200 |
More specifically:
- try without numeric-tail only if LFN didn't have invalid short chars
- start at ~1 (instead of ~0)
- handle case if numeric tail is more than one char (ie > 10)
Windows 9x Scandisk doesn't see anymore mismatches between short file names and
long file names for non-ASCII filenames.
Specification: "FAT: General overview of on-disk format" v1.03, page 31
Signed-off-by: Hervé Poussineau <address@hidden>
---
block/vvfat.c | 62 ++++++++++++++++++++++++++++-------------------------------
1 file changed, 29 insertions(+), 33 deletions(-)
diff --git a/block/vvfat.c b/block/vvfat.c
index 3cb65493cd..414bca6dee 100644
--- a/block/vvfat.c
+++ b/block/vvfat.c
@@ -529,7 +529,8 @@ static uint8_t to_valid_short_char(gunichar c)
}
static direntry_t *create_short_filename(BDRVVVFATState *s,
- const char *filename)
+ const char *filename,
+ unsigned int directory_start)
{
int i, j = 0;
direntry_t *entry = array_get_next(&(s->directory));
@@ -587,8 +588,32 @@ static direntry_t *create_short_filename(BDRVVVFATState *s,
}
}
}
- (void)lossy_conversion;
- return entry;
+
+ /* numeric-tail generation */
+ for (j = 0; j < 8; j++) {
+ if (entry->name[j] == ' ') {
+ break;
+ }
+ }
+ for (i = lossy_conversion ? 1 : 0; i < 999999; i++) {
+ direntry_t *entry1;
+ if (i > 0) {
+ int len = sprintf(tail, "~%d", i);
+ memcpy(entry->name + MIN(j, 8 - len), tail, len);
+ }
+ for (entry1 = array_get(&(s->directory), directory_start);
+ entry1 < entry; entry1++) {
+ if (!is_long_name(entry1) &&
+ !memcmp(entry1->name, entry->name, 11)) {
+ break; /* found dupe */
+ }
+ }
+ if (entry1 == entry) {
+ /* no dupe found */
+ return entry;
+ }
+ }
+ return NULL;
}
/* fat functions */
@@ -701,36 +726,7 @@ static inline direntry_t*
create_short_and_long_name(BDRVVVFATState* s,
}
entry_long=create_long_filename(s,filename);
- entry = create_short_filename(s, filename);
-
- /* mangle duplicates */
- while(1) {
- direntry_t* entry1=array_get(&(s->directory),directory_start);
- int j;
-
- for(;entry1<entry;entry1++)
- if(!is_long_name(entry1) && !memcmp(entry1->name,entry->name,11))
- break; /* found dupe */
- if(entry1==entry) /* no dupe found */
- break;
-
- /* use all 8 characters of name */
- if(entry->name[7]==' ') {
- int j;
- for(j=6;j>0 && entry->name[j]==' ';j--)
- entry->name[j]='~';
- }
-
- /* increment number */
- for(j=7;j>0 && entry->name[j]=='9';j--)
- entry->name[j]='0';
- if(j>0) {
- if(entry->name[j]<'0' || entry->name[j]>'9')
- entry->name[j]='0';
- else
- entry->name[j]++;
- }
- }
+ entry = create_short_filename(s, filename, directory_start);
/* calculate checksum; propagate to long name */
if(entry_long) {
--
2.11.0
- Re: [Qemu-block] [Qemu-devel] [PATCH v2 13/13] vvfat: change OEM name to 'MSWIN4.1', (continued)
[Qemu-block] [PATCH v2 09/13] vvfat: correctly create base short names for non-ASCII filenames, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 11/13] vvfat: limit number of entries in root directory in FAT12/FAT16, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 01/13] vvfat: fix qemu-img map and qemu-img convert, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 07/13] vvfat: always create . and .. entries at first and in that order, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 03/13] vvfat: fix typos, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 10/13] vvfat: correctly generate numeric-tail of short file names,
Hervé Poussineau <=
[Qemu-block] [PATCH v2 04/13] vvfat: rename useless enumeration values, Hervé Poussineau, 2017/05/22
[Qemu-block] [PATCH v2 02/13] vvfat: replace tabs by 8 spaces, Hervé Poussineau, 2017/05/22
Re: [Qemu-block] [Qemu-devel] [PATCH v2 00/13] vvfat: misc fixes for read-only mode, no-reply, 2017/05/22