Re: [patch] cifs: return more accurate errno

On Tue, May 18, 2010 at 12:18 PM, Steve French <[email protected]> wrote:
> I will check.
>
> On Tue, May 18, 2010 at 7:06 AM, Jiri Kosina <[email protected]> wrote:
>> On Fri, 7 May 2010, Dan Carpenter wrote:
>>
>>> Smatch compains that we don't use the return value from get_dfs_path().
>>>
>>> In the original code if get_dfs_path() fails we return ERR_PTR(-ENOENT),
>>> but with this patch we can return errno from get_dfs_path() directly.
>>>
>>> Signed-off-by: Dan Carpenter <[email protected]>

ENOENT does seem like a safe return code if a DFS referral can not be
followed, but if the intent was to remap all error cases here to ENOENT
it would be easier to follow with something similar to your code e.g.

+       if (rc < 0) {
+               rc = -ENOENT;
+               goto out_err;
+      }

Igor,
Did you intend to always return ENOENT for DFS when failures looking up
DFS referrals (probably most likely to be either error coming back from
 tree connect to IPC$ or error from the call to GetDFSRefer)?

-- 
Thanks,

Steve

)