Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Kohada.Tetsuhiro@dc.MitsubishiElectric.co.jp" 
        <Kohada.Tetsuhiro@dc.MitsubishiElectric.co.jp>
To:     'Vasant Karasulli' <vkarasulli@suse.de>
CC:     'Sungjong Seo' <sj1557.seo@samsung.com>,
        "'linux-fsdevel@vger.kernel.org'" <linux-fsdevel@vger.kernel.org>,
        "'linux-kernel@vger.kernel.org'" <linux-kernel@vger.kernel.org>,
        'Takashi Iwai' <tiwai@suse.de>,
        'David Disseldorp' <ddiss@suse.de>,
        'Namjae Jeon' <linkinjeon@kernel.org>
Subject: RE: [PATCH v2 2/2] exfat currently unconditionally strips trailing
 periods '.' when performing path lookup, but allows them in the filenames
 during file creation. This is done intentionally, loosely following Windows
 behaviour and specifications which ...
Thread-Topic: [PATCH v2 2/2] exfat currently unconditionally strips trailing
 periods '.' when performing path lookup, but allows them in the filenames
 during file creation. This is done intentionally, loosely following Windows
 behaviour and specifications which ...
Thread-Index: AQHYNLplcevRMz/jOUuuvmG0JvLRkKy55GyAgAAFMQCABE5KoA==
Date:   Mon, 14 Mar 2022 03:52:08 +0000
Deferred-Delivery: Mon, 14 Mar 2022 03:54:00 +0000
Message-ID: <TYAPR01MB535314A6E1FB0CB1BAD621C2900F9@TYAPR01MB5353.jpnprd01.prod.outlook.com>
References: <20220310142455.23127-1-vkarasulli@suse.de>
 <20220310142455.23127-3-vkarasulli@suse.de> <20220310210633.095f0245@suse.de>
 <CAKYAXd_ij3WqJHQZvH458XRwLBtboiJnr-fK0hVPDi_j_8XDZQ@mail.gmail.com>
 <YisU2FA7EBeguwN5@vasant-suse>
In-Reply-To: <YisU2FA7EBeguwN5@vasant-suse>
Accept-Language: ja-JP, en-US
Content-Language: ja-JP
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-2022-jp?B?b0MxRk9sbFhlQlZhalloMis4NGQveGdyME1lMURIRkVrTC9CcjhxcDgv?=
 =?iso-2022-jp?B?cngxTWhVZWdMUXFqUVVYaUpzZmR6QUsrNTJwYVhDWDIyRWRoRk42QXNk?=
 =?iso-2022-jp?B?S00zRXprUlJHZi8vWVRZbnBQd2t3TDRjenlRTUNiNDB0emdJMkdJVVlr?=
 =?iso-2022-jp?B?b2FrL3d0UzIwNmltOGw2MGE3Rkc0cHB1RnBZVnBnYkppMWRZdzBSMjd4?=
 =?iso-2022-jp?B?dVJwZEdhWFAxYzhuVGdNb2FNS1ovODFxQ09LS09tbWJsWGFjcUh3SlEx?=
 =?iso-2022-jp?B?bDUyZDExOWhJT3dvM1VrUmZjcy9VbnhaMVNrUnJHZkl5RHBYaUkrR0wz?=
 =?iso-2022-jp?B?UHhwaGFVRk1Fbkh2V29SRmovZjdVWTg4cmcyQ2twdEptTGVPRHhQbEYz?=
 =?iso-2022-jp?B?SXNvZVdKc3VoaXZTVTZtbVUvWlJhVjB2M3p6UDFMcEFhMUp4bDZuMStL?=
 =?iso-2022-jp?B?NytZaytvL05QTDUzNk5WaXo1MndudHlMZURUejZna3RSL1JjcWdFZkhk?=
 =?iso-2022-jp?B?QzB2cS8xQzVOMThKK3FDTk56YXEwU29BQmdBcEEwNFM1cVlKb1VkZ0J0?=
 =?iso-2022-jp?B?ajMxVWRYbU9LaGpzQzFMT0VKRzZub3FPdE1mNWhqOWl4b1ZLNFREMlQz?=
 =?iso-2022-jp?B?M3czK2lWaTBYVlo3UWU0UUZ6NjFJTGcvV0hZTXFiT0tKQ2dtVUFpNkdS?=
 =?iso-2022-jp?B?MUwvL3RVN2RaaTNsL0xKTzAyaWZLUEFFV1l2WGNQNzFwOVA3ZzZyMFVa?=
 =?iso-2022-jp?B?OWM5SkRuMFdLUFdiKzhjMWxFdE1zRWc0bURnMW1jNVdDempDY1BINDJN?=
 =?iso-2022-jp?B?UGRqdzJQUUVocDlMVjd5aVdlbVAreEdzTnY4K3MzRGpscXA0dDdsK0pz?=
 =?iso-2022-jp?B?TENkZStGZGNRZ1lTNU9oaWZPLzZUK2VzSnZabUU4K0c4NWdtM05SSVFu?=
 =?iso-2022-jp?B?ZHU0ekMvM2ZML2pEL3FKWXNDUmsvZmw5cEIvNEVDOVREZDBOVkVMQ2Zv?=
 =?iso-2022-jp?B?K0RaQVJoRkhrTVFuOWFHRHVtUnh0dWRSbnJ2NXFraXRaandOWHJHMjdQ?=
 =?iso-2022-jp?B?ZUdqbVBaekpCaXFWeWUzV3VKL0ZyWS9aM3R4U3dXcWF6OURHQmNxOEc2?=
 =?iso-2022-jp?B?dXhieWJ2NzkvRUhhODIxYXpabzl4KzJCaDl5QkNTTG95cU5KcVpRWkF5?=
 =?iso-2022-jp?B?aHBVRjBkYzNFSEszL1ZCTXhSR0l5VENBOWZkMjN2SHlyZkQ3UGk5WEFz?=
 =?iso-2022-jp?B?YXRwbjUyd3NMSjdIUmRVQVBCbU5lTUxRdGJmcGVUa0hMcUZsN2tmTmt1?=
 =?iso-2022-jp?B?elBBQ2E1RS9rWEVKTHVBRHRhY3hNWVFidmVIUmhMaDlOVmdQNXFlVHVD?=
 =?iso-2022-jp?B?NHN1a3pVNXpPTUc3SmR2RVlBQVFsZjFGWjVwRFlhVTdkMW5IbTZySHRH?=
 =?iso-2022-jp?B?NjdLb2lNcWZiTkVydUhBeW91emFrUENDQmdYK3ZMbXVyQnlNSTFpVmJn?=
 =?iso-2022-jp?B?cXRHTnJjdFlTbk5EUUdHMjZJSk9TYTFmblV6NXg2Z3c4SHlPamIzZE9N?=
 =?iso-2022-jp?B?ZVJEcEIvOU9WUVRRWVV1V3laOVZCREVnbmZWcGNuNXNMV3gxSmlCaE9z?=
 =?iso-2022-jp?B?dDA4Q0Y3WmRWSXdJakdldXdHZEZLYUN0cEJGSEt3bEZ4NDdRN3JQbTd3?=
 =?iso-2022-jp?B?N2F3RzhoK1pERmN1Q3VMZzdHb3JlUk81b2xjQTdoR1dRZDFaMXZmU3Y3?=
 =?iso-2022-jp?B?TGNWZHhHY3ZxQW1heU9ZOS9ZRlBPdk9tRi9WeDJ3NTBiRFJGd3lDak1U?=
 =?iso-2022-jp?B?WFUzZGRkOUpYNk5kZy9TQlNYTmdQR210NC9DWE1ORDc2cXVXMVZPdjdl?=
 =?iso-2022-jp?B?eW9sZ0FlR0wxRk9iSkFNMWdLNlJWVitRcGRLZzZGK3FQU2h3eUNkdGZV?=
 =?iso-2022-jp?B?MHFOemZ5SjUxWHplcTRzUGI0MXhPYWFuekJ6clFyMkdzQm0zSmZseUNX?=
 =?iso-2022-jp?B?Nlo5R0U1YWhha09aWWdpdFpGN2o5Vk82cUlYNkNYdXh0M2FsRUpUVWgy?=
 =?iso-2022-jp?B?aktjbHdVRDBRV3pXN0JxejY1SHZtZmhvU011cWtDRjhuRjJXZytHMG1h?=
 =?iso-2022-jp?B?dUY=?=
Content-Type: text/plain; charset="iso-2022-jp"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: dc.MitsubishiElectric.co.jp
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: TYAPR01MB5353.jpnprd01.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a1d493fd-15cd-4d73-05a6-08da056e49bf
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Mar 2022 03:54:06.7518
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: c5a75b62-4bff-4c96-a720-6621ce9978e5
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: jTxZi19/3BDHiv/XY6r2P4wPgMxjDIPlOaQ1artr4ZPFxAUuDsq9UouOEoIZei0qDuH+2D8/KW9mNVuQw3lsiw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OS3PR01MB6039
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi, Vasant Karasulli.

> > > I think it makes sense to mention your findings from the Windows
> > > tests here. E.g. "Windows 10 also retains leading and trailing space
> > > characters".
> > Windows 10 do also strip them. So you can make another patch to strip
> > it as well as trailing periods.
> Actually I found contradicting behavior between Window 10 File Explorer a=
nd Commandline. Commandline seems to strip
> trailing spaces, but File Explorer doesn't.

The exfat specification specifies an invalid character set, but there are n=
o restrictions on the use of leading or trailing white-space or dots.
Even if the filename has trailing-dot as shown below, it conforms to the ex=
fat specification and can be created on Windows.
"a"
"a."
"a.."
These are treated as "a" in the current implementation of linix-exfat, so t=
he intended file cannot be accessed.
The specified filename should not be modified to comply with the exfat spec=
ification.
Therefore, exfat_striptail_len() should not be used.

Note:
Windows explorer removes trailing white-space and dots, but not the behavio=
r of the filesystem.
Also, you can create a trailing-dot filename by quoting it on the command l=
ine.

BR
T.Kohada