sys_uselib in exec.c

[Shaya Potter <spotter@cs.columbia.edu>]

the code there is (2.6.1)

125         error = __user_walk(library, LOOKUP_FOLLOW|LOOKUP_OPEN,
&nd);
126         if (error)
127                 goto out;
128 
129         error = -EINVAL;
130         if (!S_ISREG(nd.dentry->d_inode->i_mode))
131                 goto exit;
132 
133         error = permission(nd.dentry->d_inode, MAY_READ | MAY_EXEC,
&nd);
134         if (error)
135                 goto exit;
136 
137         file = dentry_open(nd.dentry, nd.mnt, O_RDONLY);
138         error = PTR_ERR(file);
139         if (IS_ERR(file))
140                 goto out;

where exit does a path_release(&nd) and return's error while out just
returns error.

if dentry_open fails, we don't call path_release(&nd).  Is this correct?

thanks,

shaya