I would like to report a bug in the Samba while adding an entry in the [stat_cache_add](https://gitlab.com/samba-team/samba/-/blob/master/source3/smbd/statcache.c#L74-91) method, the idea of this conditional block
 
/*
        * If we are in case insentive mode, we don't need to
        * store names that need no translation - else, it
        * would be a waste.
        */
 
        if (!case_sensitive && (strcmp(full_orig_name, translated_path) == 0)) {
               TALLOC_FREE(translated_path);
               return;
        }
 
Is if the name of the file on filesystem and the requested path are same, then no need to cache in case-insensitive mode. This [commit](https://github.com/samba-team/samba/commit/6e364c545384c5da4a0f0f0536c40394aa2e2a97)
 Prefixed the translated name with TWRP token while the original requested filename is not prefixed with the TWRP token. So, the condition is failing after the commit even if the requested filename and the filename on the filesystem match. I think either it should done before adding the TWRP token or, add the TWRP token in the original filename before making the comparison.

With this change in beahvior this is the impact. 

There are two files on the fielsystem:

1. foo.txt with contents foo
2. FOO.txt with contents FOO

In Samba 4.11, 

1. Get-Content S:\foo.txt  returns "foo"
2. Get-Content S:\FOO.txt returns "FOO"

In Samba 4.14,

1. Get-Content S:\foo.txt  returns "foo"
2. Get-Content S:\FOO.txt returns "foo"