CIFS共享目录挂载后执行mv失败

【严重程度】 一般
【特性】 文件系统
【重现类型】 有条件必然重现
【定位分析】
~~~
在共享目录下，对已有文件bbb重命名为aaa，必定出现报错No such file or directory
根据代码分析 + 抓包验证
发现5.10 和 4.18 发送的请求数据的不同之处在于：
\likz\\likz001-850\bbb  -> 5.10
\likz\likz001-850\bbb   -> 4.18
可见多了一个 '/' 路径符号
5.10客户端在服务端mv 操作分三步，比如mv bbb aaa：
查找bbb是否存在，请求路径：\likz\\likz001-850\bbb
查找aaa是否存在，请求路径：\likz\\likz001-850\aaa
执行重命名操作，请求路径名：\likz\\likz001-850\bbb 和 \likz\\likz001-850\aaa
既然步骤1和步骤2都能够按照预期执行，也就说明这个路径可以解析处理，
但是服务端没有解析，因此服务端存在潜在BUG，

4.18没有此现象，但5.10有问题，根因分析：
rename流程：
cifs_rename2
  build_path_from_dentry
    build_path_from_dentry_optional_prefix
build_path_from_dentry_optional_prefix(struct dentry *direntry, bool prefix)
{
    struct dentry *temp;
    .....
    char *full_path;
    .....
    
    if (cifs_sb->mnt_cifs_flags & CIFS_MOUNT_USE_PREFIX_PATH)
        pplen = cifs_sb->prepath ? strlen(cifs_sb->prepath) + 1 : 0;   【检查点  1】
    
    // 找到根dentry: temp，统计出full_path 长度// 根据根dentry 反向创建路径 full_path=/aaa/bbb ,最后一个路径不带‘\’if (pplen) {  // 如果【检查点 1】，即使用前缀路径，   【检查点  2】int i;

cifs_dbg(FYI, "using cifs_sb prepath <%s>\n", cifs_sb->prepath);
        memcpy(full_path+dfsplen+1, cifs_sb->prepath, pplen-1);
        full_path[dfsplen] = dirsep;  // 第一个位置加 ‘\’for (i = 0; i < pplen-1; i++)
            if (full_path[dfsplen+1+i] == '/')  // 如果是‘/’，就替换为‘\’
                full_path[dfsplen+1+i] = CIFS_DIR_SEP(cifs_sb); 【检查点 3】
    }
}
5.10 版本内核在 cifs_mount 流程中，强制设置了“CIFS_MOUNT_USE_PREFIX_PATH”
即强制使用前缀路径prepath
int cifs_mount(struct cifs_sb_info *cifs_sb, struct smb_vol *vol)
{
    ......
    cifs_sb->mnt_cifs_flags |= CIFS_MOUNT_USE_PREFIX_PATH;
    ......
}
补丁：https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.2-rc6&id=bacd704a95ad0b93af995aae4a523aa046f46563
按照作者的意思，这样可以在遇到错误重新连接的过程中简化流程。
因此5.10 版本内核会进入【检查点 2】 位置，而
然而，客户端内核在创建prepath，并不会清理掉 mount.cifs 命令中最后自带的'/'.
最终导致'/'在【检查点 3】被替换为'',再拼接temp创建的路径，得到\\。
服务端在rename时可能忘了处理\\的情况，导致错误发生
4.18 & 4.19 & 3.10虽然不强制使用前缀路径，但是如果因服务端路径异常，仍然有可能使用前缀路径，仍然有触发该问题的风险，
创建prepath的函数：cifs_parse_devname()、
cifs_parse_devname(const char *devname, struct smb_vol *vol)
{
    ......
    // 直接复制用户态传入的路径
    vol->prepath = kstrdup(pos, GFP_KERNEL);
}
因此，需要对用户传入的路径进行处理。
主线补丁：https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bacd704a95ad0b93af995aae4a523aa046f46563 
对用户传入路径进行过滤，去除了多余的/，解决了在重命名过程中，服务端报错的问题。

综上，该问题的根因：CIFS客户端没有对用户态输入的路径字符进行过滤

hulk-7.5、hulk-8.1、hulk-5.10、hulk-4.19均存在未对路径进行过滤的问题。

因此涉及版本都应当回合该修复补丁。
~~~
【影响评估及测试建议】 可能会影响到下游CIFS的正常使用</p><p>
【对外部影响描述】 无</p><p>

GVP openEuler/kernel
Closed

Content Risk Flag

Comments (1)

GVPopenEuler/kernelClosed

Content Risk Flag