From mboxrd@z Thu Jan  1 00:00:00 1970
From: Rob Clark <robdclark@gmail.com>
Date: Sun, 24 Sep 2017 16:27:22 -0400
Subject: [U-Boot] FAT filesystems and mtools-created filesystems
In-Reply-To: <d5575419-4ce0-8218-63bc-e9e8dc810a33@iki.fi>
References: <daa2a636-6e9b-dcd3-bc78-d4a8e24494f8@iki.fi>
 <d5575419-4ce0-8218-63bc-e9e8dc810a33@iki.fi>
Message-ID: <CAF6AEGvRj6URL25Gj1R-pxwXToOC-hX_GQfqUmsCqK5zUMi4Dw@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sat, Sep 23, 2017 at 6:51 AM, Tuomas Tynkkynen
<tuomas.tynkkynen@iki.fi> wrote:
> On 09/23/2017 01:26 PM, Tuomas Tynkkynen wrote:
>>
>> Hi,
>>
>> FAT file systems created by GNU mtools have a problem that mtools doesn't
>> initialize the first cluster field of the '.' and '..' directory entries.
>> That is, with the following script:

Sorry for the delay, I've been travelling.

I'll have a look at this.. leaning towards moving sanitize_path() from
efi_file.c into fs/fat, since we don't have an infinite stack to store
parent ".." entries.

Definitely would be good if you could send a patch to add this to
test/fs/fs-test.sh

>>
>
> Oh, and unrelated to the mtools problem, '..' entries leading to the root
> directory need fixing in the current code, as itr->is_root needs to be set.

Hmm, I thought I fixed this once, but I guess only for fat32..

BR,
-R