gfs2: Remove duplicate i_nlink check from gfs2_link()
authorAndrew Price <anprice@redhat.com>
Tue, 14 Mar 2023 13:18:27 +0000 (13:18 +0000)
committerAndreas Gruenbacher <agruenba@redhat.com>
Mon, 27 Mar 2023 12:39:47 +0000 (14:39 +0200)
commit8dc14966ca3eb5bf4e200c50cc73199ee6de2bd7
treeb6375834dd96b339a0f8c02bef445a897396fb90
parent1e760fa3596e8c7f08412712c168288b79670d78
gfs2: Remove duplicate i_nlink check from gfs2_link()

The duplication is:

    struct gfs2_inode *ip = GFS2_I(inode);
    [...]
    error = -ENOENT;
    if (inode->i_nlink == 0)
        goto out_gunlock;
    [...]
    error = -EINVAL;
    if (!ip->i_inode.i_nlink)
        goto out_gunlock;

The second check is removed. ENOENT is the correct error code for
attempts to link a deleted inode (ref: link(2)).

If we support O_TMPFILE in future the check will need to be updated with
an exception for inodes flagged I_LINKABLE so sorting out this
duplication now will make it a slightly cleaner change.

Signed-off-by: Andrew Price <anprice@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
fs/gfs2/inode.c