From patchwork Mon May 15 20:31:09 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Herv=C3=A9_Poussineau?= X-Patchwork-Id: 762703 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3wRXQL1126z9s0m for ; Tue, 16 May 2017 06:35:22 +1000 (AEST) Received: from localhost ([::1]:38638 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dAMi7-000415-QO for incoming@patchwork.ozlabs.org; Mon, 15 May 2017 16:35:19 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35062) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dAMeP-0001Pe-7J for qemu-devel@nongnu.org; Mon, 15 May 2017 16:31:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dAMeN-0003Xo-IG for qemu-devel@nongnu.org; Mon, 15 May 2017 16:31:29 -0400 Received: from smtp2-g21.free.fr ([2a01:e0c:1:1599::11]:22375) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dAMeJ-0003UF-Kr; Mon, 15 May 2017 16:31:23 -0400 Received: from localhost.localdomain (unknown [82.227.227.196]) by smtp2-g21.free.fr (Postfix) with ESMTP id 366842003F8; Mon, 15 May 2017 22:31:22 +0200 (CEST) From: =?UTF-8?q?Herv=C3=A9=20Poussineau?= To: qemu-devel@nongnu.org Date: Mon, 15 May 2017 22:31:09 +0200 Message-Id: <20170515203114.9477-10-hpoussin@reactos.org> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170515203114.9477-1-hpoussin@reactos.org> References: <20170515203114.9477-1-hpoussin@reactos.org> MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: Windows NT kernel [generic] [fuzzy] X-Received-From: 2a01:e0c:1:1599::11 Subject: [Qemu-devel] [PATCH 09/13] vvfat: correctly create base short names for non-ASCII filenames X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , =?UTF-8?q?Herv=C3=A9=20Poussineau?= , qemu-block@nongnu.org, Max Reitz Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" More specifically, create short name from long name and change blacklist of invalid chars to whitelist of valid chars. Windows 9x also now correctly see long file names of filenames containing a space, but Scandisk still complains about mismatch between SFN and LFN. Specification: "FAT: General overview of on-disk format" v1.03, pages 30/31 Signed-off-by: Hervé Poussineau --- block/vvfat.c | 84 +++++++++++++++++++++++++++++++++++++++-------------------- 1 file changed, 56 insertions(+), 28 deletions(-) diff --git a/block/vvfat.c b/block/vvfat.c index 5f6356c834..37cfaa85fc 100644 --- a/block/vvfat.c +++ b/block/vvfat.c @@ -589,6 +589,60 @@ static void set_begin_of_direntry(direntry_t* direntry, uint32_t begin) direntry->begin_hi = cpu_to_le16((begin >> 16) & 0xffff); } +static uint8_t to_valid_short_char(uint8_t c) +{ + if ((c >= '0' && c <= '9') || + (c >= 'A' && c <= 'Z') || + strchr("$%'-_@~`!(){}^#&", c) != 0) { + return c; + } else if (c >= 'a' && c <= 'z') { + return c - 'a' + 'A'; + } else { + return 0; + } +} + +static direntry_t *create_short_filename(BDRVVVFATState *s, long_file_name *lfn) +{ + int i, j = 0; + direntry_t *entry = array_get_next(&(s->directory)); + uint8_t c; + + if (!entry) { + return NULL; + } + memset(entry->name, 0x20, sizeof(entry->name)); + + /* create short name (keep only valid chars and upcase letters) */ + for (i = 0; i < lfn->len && j < 8; i += 2) { + c = to_valid_short_char(lfn->name[i]); + if (lfn->name[i + 1] == 0 && c != 0) { + entry->name[j++] = c; + } else if (lfn->name[i + 1] == 0 && + (lfn->name[i] == '.' || lfn->name[i] == 0)) { + break; + } + } + /* search last extension */ + for (i = lfn->len - 2; i > 0; i -= 2) { + if (lfn->name[i] == '.' && lfn->name[i + 1] == 0) { + break; + } + } + if (i != 0) { + /* create short extension */ + for (; i < lfn->len && j < 11; i += 2) { + c = to_valid_short_char(lfn->name[i]); + if (lfn->name[i + 1] == 0 && c != 0) { + entry->name[j++] = c; + } else if (lfn->name[i + 1] == 0 && lfn->name[i] == 0) { + break; + } + } + } + return entry; +} + /* fat functions */ static inline uint8_t fat_chksum(const direntry_t* entry) @@ -687,7 +741,7 @@ static inline void init_fat(BDRVVVFATState* s) static inline direntry_t* create_short_and_long_name(BDRVVVFATState* s, unsigned int directory_start, const char* filename, int is_dot) { - int i,j,long_index=s->directory.next; + int long_index = s->directory.next; direntry_t* entry = NULL; direntry_t* entry_long = NULL; long_file_name lfn; @@ -700,33 +754,7 @@ static inline direntry_t* create_short_and_long_name(BDRVVVFATState* s, } entry_long = create_long_filename(s, filename, &lfn); - - i = strlen(filename); - for(j = i - 1; j>0 && filename[j]!='.';j--); - if (j > 0) - i = (j > 8 ? 8 : j); - else if (i > 8) - i = 8; - - entry=array_get_next(&(s->directory)); - memset(entry->name, 0x20, sizeof(entry->name)); - memcpy(entry->name, filename, i); - - if (j > 0) { - for (i = 0; i < 3 && filename[j + 1 + i]; i++) { - entry->name[8 + i] = filename[j + 1 + i]; - } - } - - /* upcase & remove unwanted characters */ - for(i=10;i>=0;i--) { - if(i==10 || i==7) for(;i>0 && entry->name[i]==' ';i--); - if(entry->name[i]<=' ' || entry->name[i]>0x7f - || strchr(".*?<>|\":/\\[];,+='",entry->name[i])) - entry->name[i]='_'; - else if(entry->name[i]>='a' && entry->name[i]<='z') - entry->name[i]+='A'-'a'; - } + entry = create_short_filename(s, &lfn); /* mangle duplicates */ while(1) {