Hi,
Stefan Metzmacher <metze@xxxxxxxxx> writes:
>> + if (tcon->ipc) {
>> + scnprintf(tree, MAX_TREE_SIZE, "\\\\%s\\IPC$", share);
>> + rc = ops->tree_connect(xid, tcon->ses, tree, tcon, nlsc);
>> + } else {
>> + scnprintf(tree, MAX_TREE_SIZE, "\\%s", share);
>> + rc = ops->tree_connect(xid, tcon->ses, tree, tcon, nlsc);
>> + if (!rc) {
>> + rc = update_super_prepath(tcon, prefix, prefix_len);
>> + break;
>> + }
>> + }
>
> The original code used:
>
> - scnprintf(tree, MAX_TREE_SIZE, "\\%.*s", (int)share_len,
> - share);
>
> So it's no longer identical.
>
> Was the share_len code added after my initial patch?
Yep. See
e4af35fa55b0 (cifs: handle hostnames that resolve to same ip in failover)
bacd704a95ad (cifs: handle prefix paths in reconnect)
