Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (c) 2000-2006 Silicon Graphics, Inc. |
| 3 | * Copyright (c) 2012-2013 Red Hat, Inc. |
| 4 | * All rights reserved. |
| 5 | * |
| 6 | * This program is free software; you can redistribute it and/or |
| 7 | * modify it under the terms of the GNU General Public License as |
| 8 | * published by the Free Software Foundation. |
| 9 | * |
| 10 | * This program is distributed in the hope that it would be useful, |
| 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| 13 | * GNU General Public License for more details. |
| 14 | * |
| 15 | * You should have received a copy of the GNU General Public License |
| 16 | * along with this program; if not, write the Free Software Foundation, |
| 17 | * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA |
| 18 | */ |
| 19 | #include "xfs.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 20 | #include "xfs_shared.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 21 | #include "xfs_fs.h" |
Dave Chinner | 6ca1c90 | 2013-08-12 20:49:26 +1000 | [diff] [blame] | 22 | #include "xfs_format.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 23 | #include "xfs_log_format.h" |
| 24 | #include "xfs_trans_resv.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 25 | #include "xfs_bit.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 26 | #include "xfs_mount.h" |
Dave Chinner | 5706278 | 2013-10-15 09:17:51 +1100 | [diff] [blame] | 27 | #include "xfs_da_format.h" |
Dave Chinner | d6cf130 | 2014-06-06 15:14:11 +1000 | [diff] [blame] | 28 | #include "xfs_da_btree.h" |
Darrick J. Wong | 3ab78df | 2016-08-03 11:15:38 +1000 | [diff] [blame] | 29 | #include "xfs_defer.h" |
Dave Chinner | 2b9ab5a | 2013-08-12 20:49:37 +1000 | [diff] [blame] | 30 | #include "xfs_dir2.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 31 | #include "xfs_inode.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 32 | #include "xfs_ialloc.h" |
| 33 | #include "xfs_alloc.h" |
| 34 | #include "xfs_bmap.h" |
Dave Chinner | a4fbe6a | 2013-10-23 10:51:50 +1100 | [diff] [blame] | 35 | #include "xfs_bmap_btree.h" |
Dave Chinner | 6898811 | 2013-08-12 20:49:42 +1000 | [diff] [blame] | 36 | #include "xfs_bmap_util.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 37 | #include "xfs_error.h" |
| 38 | #include "xfs_quota.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 39 | #include "xfs_trans_space.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 40 | #include "xfs_trace.h" |
| 41 | #include "xfs_symlink.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 42 | #include "xfs_trans.h" |
Dave Chinner | 239880e | 2013-10-23 10:50:10 +1100 | [diff] [blame] | 43 | #include "xfs_log.h" |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 44 | |
| 45 | /* ----- Kernel only functions below ----- */ |
Darrick J. Wong | 5da8f2f | 2017-06-16 11:00:15 -0700 | [diff] [blame] | 46 | int |
| 47 | xfs_readlink_bmap_ilocked( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 48 | struct xfs_inode *ip, |
| 49 | char *link) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 50 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 51 | struct xfs_mount *mp = ip->i_mount; |
| 52 | struct xfs_bmbt_irec mval[XFS_SYMLINK_MAPS]; |
| 53 | struct xfs_buf *bp; |
| 54 | xfs_daddr_t d; |
| 55 | char *cur_chunk; |
| 56 | int pathlen = ip->i_d.di_size; |
| 57 | int nmaps = XFS_SYMLINK_MAPS; |
| 58 | int byte_cnt; |
| 59 | int n; |
| 60 | int error = 0; |
| 61 | int fsblocks = 0; |
| 62 | int offset; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 63 | |
Christoph Hellwig | 29db250 | 2017-07-13 12:14:34 -0700 | [diff] [blame] | 64 | ASSERT(xfs_isilocked(ip, XFS_ILOCK_SHARED | XFS_ILOCK_EXCL)); |
| 65 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 66 | fsblocks = xfs_symlink_blocks(mp, pathlen); |
| 67 | error = xfs_bmapi_read(ip, 0, fsblocks, mval, &nmaps, 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 68 | if (error) |
| 69 | goto out; |
| 70 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 71 | offset = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 72 | for (n = 0; n < nmaps; n++) { |
| 73 | d = XFS_FSB_TO_DADDR(mp, mval[n].br_startblock); |
| 74 | byte_cnt = XFS_FSB_TO_B(mp, mval[n].br_blockcount); |
| 75 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 76 | bp = xfs_buf_read(mp->m_ddev_targp, d, BTOBB(byte_cnt), 0, |
| 77 | &xfs_symlink_buf_ops); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 78 | if (!bp) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 79 | return -ENOMEM; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 80 | error = bp->b_error; |
| 81 | if (error) { |
| 82 | xfs_buf_ioerror_alert(bp, __func__); |
| 83 | xfs_buf_relse(bp); |
Dave Chinner | ac75a1f | 2014-03-07 16:19:14 +1100 | [diff] [blame] | 84 | |
| 85 | /* bad CRC means corrupted metadata */ |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 86 | if (error == -EFSBADCRC) |
| 87 | error = -EFSCORRUPTED; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 88 | goto out; |
| 89 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 90 | byte_cnt = XFS_SYMLINK_BUF_SPACE(mp, byte_cnt); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 91 | if (pathlen < byte_cnt) |
| 92 | byte_cnt = pathlen; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 93 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 94 | cur_chunk = bp->b_addr; |
| 95 | if (xfs_sb_version_hascrc(&mp->m_sb)) { |
Eric Sandeen | bda65ef | 2014-04-14 19:05:43 +1000 | [diff] [blame] | 96 | if (!xfs_symlink_hdr_ok(ip->i_ino, offset, |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 97 | byte_cnt, bp)) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 98 | error = -EFSCORRUPTED; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 99 | xfs_alert(mp, |
| 100 | "symlink header does not match required off/len/owner (0x%x/Ox%x,0x%llx)", |
| 101 | offset, byte_cnt, ip->i_ino); |
| 102 | xfs_buf_relse(bp); |
| 103 | goto out; |
| 104 | |
| 105 | } |
| 106 | |
| 107 | cur_chunk += sizeof(struct xfs_dsymlink_hdr); |
| 108 | } |
| 109 | |
Eric Sandeen | 2ac56d3 | 2015-06-22 09:42:48 +1000 | [diff] [blame] | 110 | memcpy(link + offset, cur_chunk, byte_cnt); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 111 | |
| 112 | pathlen -= byte_cnt; |
| 113 | offset += byte_cnt; |
| 114 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 115 | xfs_buf_relse(bp); |
| 116 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 117 | ASSERT(pathlen == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 118 | |
| 119 | link[ip->i_d.di_size] = '\0'; |
| 120 | error = 0; |
| 121 | |
| 122 | out: |
| 123 | return error; |
| 124 | } |
| 125 | |
| 126 | int |
| 127 | xfs_readlink( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 128 | struct xfs_inode *ip, |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 129 | char *link) |
| 130 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 131 | struct xfs_mount *mp = ip->i_mount; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 132 | xfs_fsize_t pathlen; |
| 133 | int error = 0; |
| 134 | |
| 135 | trace_xfs_readlink(ip); |
| 136 | |
Christoph Hellwig | 30ee052 | 2016-04-06 07:53:29 +1000 | [diff] [blame] | 137 | ASSERT(!(ip->i_df.if_flags & XFS_IFINLINE)); |
| 138 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 139 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 140 | return -EIO; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 141 | |
| 142 | xfs_ilock(ip, XFS_ILOCK_SHARED); |
| 143 | |
| 144 | pathlen = ip->i_d.di_size; |
| 145 | if (!pathlen) |
| 146 | goto out; |
| 147 | |
Darrick J. Wong | 6eb0b8d | 2017-07-07 08:37:26 -0700 | [diff] [blame] | 148 | if (pathlen < 0 || pathlen > XFS_SYMLINK_MAXLEN) { |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 149 | xfs_alert(mp, "%s: inode (%llu) bad symlink length (%lld)", |
| 150 | __func__, (unsigned long long) ip->i_ino, |
| 151 | (long long) pathlen); |
| 152 | ASSERT(0); |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 153 | error = -EFSCORRUPTED; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 154 | goto out; |
| 155 | } |
| 156 | |
| 157 | |
Darrick J. Wong | 5da8f2f | 2017-06-16 11:00:15 -0700 | [diff] [blame] | 158 | error = xfs_readlink_bmap_ilocked(ip, link); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 159 | |
| 160 | out: |
| 161 | xfs_iunlock(ip, XFS_ILOCK_SHARED); |
| 162 | return error; |
| 163 | } |
| 164 | |
| 165 | int |
| 166 | xfs_symlink( |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 167 | struct xfs_inode *dp, |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 168 | struct xfs_name *link_name, |
| 169 | const char *target_path, |
| 170 | umode_t mode, |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 171 | struct xfs_inode **ipp) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 172 | { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 173 | struct xfs_mount *mp = dp->i_mount; |
| 174 | struct xfs_trans *tp = NULL; |
| 175 | struct xfs_inode *ip = NULL; |
| 176 | int error = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 177 | int pathlen; |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 178 | struct xfs_defer_ops dfops; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 179 | xfs_fsblock_t first_block; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 180 | bool unlock_dp_on_error = false; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 181 | xfs_fileoff_t first_fsb; |
| 182 | xfs_filblks_t fs_blocks; |
| 183 | int nmaps; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 184 | struct xfs_bmbt_irec mval[XFS_SYMLINK_MAPS]; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 185 | xfs_daddr_t d; |
| 186 | const char *cur_chunk; |
| 187 | int byte_cnt; |
| 188 | int n; |
| 189 | xfs_buf_t *bp; |
| 190 | prid_t prid; |
Chandra Seetharaman | 113a568 | 2013-06-27 17:25:07 -0500 | [diff] [blame] | 191 | struct xfs_dquot *udqp = NULL; |
| 192 | struct xfs_dquot *gdqp = NULL; |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 193 | struct xfs_dquot *pdqp = NULL; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 194 | uint resblks; |
| 195 | |
| 196 | *ipp = NULL; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 197 | |
| 198 | trace_xfs_symlink(dp, link_name); |
| 199 | |
| 200 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 201 | return -EIO; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 202 | |
| 203 | /* |
| 204 | * Check component lengths of the target path name. |
| 205 | */ |
| 206 | pathlen = strlen(target_path); |
Darrick J. Wong | 6eb0b8d | 2017-07-07 08:37:26 -0700 | [diff] [blame] | 207 | if (pathlen >= XFS_SYMLINK_MAXLEN) /* total string too long */ |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 208 | return -ENAMETOOLONG; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 209 | |
| 210 | udqp = gdqp = NULL; |
Zhi Yong Wu | 163467d | 2013-12-18 08:22:39 +0800 | [diff] [blame] | 211 | prid = xfs_get_initial_prid(dp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 212 | |
| 213 | /* |
| 214 | * Make sure that we have allocated dquot(s) on disk. |
| 215 | */ |
Dwight Engen | 7aab1b2 | 2013-08-15 14:08:01 -0400 | [diff] [blame] | 216 | error = xfs_qm_vop_dqalloc(dp, |
| 217 | xfs_kuid_to_uid(current_fsuid()), |
| 218 | xfs_kgid_to_gid(current_fsgid()), prid, |
| 219 | XFS_QMOPT_QUOTALL | XFS_QMOPT_INHERIT, |
| 220 | &udqp, &gdqp, &pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 221 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 222 | return error; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 223 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 224 | /* |
| 225 | * The symlink will fit into the inode data fork? |
| 226 | * There can't be any attributes so we get the whole variable part. |
| 227 | */ |
| 228 | if (pathlen <= XFS_LITINO(mp, dp->i_d.di_version)) |
| 229 | fs_blocks = 0; |
| 230 | else |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 231 | fs_blocks = xfs_symlink_blocks(mp, pathlen); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 232 | resblks = XFS_SYMLINK_SPACE_RES(mp, link_name->len, fs_blocks); |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 233 | |
| 234 | error = xfs_trans_alloc(mp, &M_RES(mp)->tr_symlink, resblks, 0, 0, &tp); |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 235 | if (error) |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 236 | goto out_release_inode; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 237 | |
Christoph Hellwig | 6552321 | 2016-11-30 14:33:25 +1100 | [diff] [blame] | 238 | xfs_ilock(dp, XFS_ILOCK_EXCL | XFS_ILOCK_PARENT); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 239 | unlock_dp_on_error = true; |
| 240 | |
| 241 | /* |
| 242 | * Check whether the directory allows new symlinks or not. |
| 243 | */ |
| 244 | if (dp->i_d.di_flags & XFS_DIFLAG_NOSYMLINKS) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 245 | error = -EPERM; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 246 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 247 | } |
| 248 | |
| 249 | /* |
| 250 | * Reserve disk quota : blocks and inode. |
| 251 | */ |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 252 | error = xfs_trans_reserve_quota(tp, mp, udqp, gdqp, |
| 253 | pdqp, resblks, 1, 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 254 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 255 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 256 | |
| 257 | /* |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 258 | * Initialize the bmap freelist prior to calling either |
| 259 | * bmapi or the directory create code. |
| 260 | */ |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 261 | xfs_defer_init(&dfops, &first_block); |
Brian Foster | 8b922f0 | 2018-05-07 17:38:48 -0700 | [diff] [blame] | 262 | tp->t_agfl_dfops = &dfops; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 263 | |
| 264 | /* |
| 265 | * Allocate an inode for the symlink. |
| 266 | */ |
| 267 | error = xfs_dir_ialloc(&tp, dp, S_IFLNK | (mode & ~S_IFMT), 1, 0, |
Chandan Rajendra | c959025 | 2018-04-02 15:47:43 -0700 | [diff] [blame] | 268 | prid, &ip); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 269 | if (error) |
| 270 | goto out_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 271 | |
| 272 | /* |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 273 | * Now we join the directory inode to the transaction. We do not do it |
| 274 | * earlier because xfs_dir_ialloc might commit the previous transaction |
| 275 | * (and release all the locks). An error from here on will result in |
| 276 | * the transaction cancel unlocking dp so don't do it explicitly in the |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 277 | * error path. |
| 278 | */ |
Christoph Hellwig | 6552321 | 2016-11-30 14:33:25 +1100 | [diff] [blame] | 279 | xfs_trans_ijoin(tp, dp, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 280 | unlock_dp_on_error = false; |
| 281 | |
| 282 | /* |
| 283 | * Also attach the dquot(s) to it, if applicable. |
| 284 | */ |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 285 | xfs_qm_vop_create_dqattach(tp, ip, udqp, gdqp, pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 286 | |
| 287 | if (resblks) |
| 288 | resblks -= XFS_IALLOC_SPACE_RES(mp); |
| 289 | /* |
| 290 | * If the symlink will fit into the inode, write it inline. |
| 291 | */ |
| 292 | if (pathlen <= XFS_IFORK_DSIZE(ip)) { |
Christoph Hellwig | 143f4ae | 2016-04-06 07:41:43 +1000 | [diff] [blame] | 293 | xfs_init_local_fork(ip, XFS_DATA_FORK, target_path, pathlen); |
| 294 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 295 | ip->i_d.di_size = pathlen; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 296 | ip->i_d.di_format = XFS_DINODE_FMT_LOCAL; |
| 297 | xfs_trans_log_inode(tp, ip, XFS_ILOG_DDATA | XFS_ILOG_CORE); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 298 | } else { |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 299 | int offset; |
| 300 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 301 | first_fsb = 0; |
| 302 | nmaps = XFS_SYMLINK_MAPS; |
| 303 | |
| 304 | error = xfs_bmapi_write(tp, ip, first_fsb, fs_blocks, |
| 305 | XFS_BMAPI_METADATA, &first_block, resblks, |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 306 | mval, &nmaps, &dfops); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 307 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 308 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 309 | |
| 310 | if (resblks) |
| 311 | resblks -= fs_blocks; |
| 312 | ip->i_d.di_size = pathlen; |
| 313 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
| 314 | |
| 315 | cur_chunk = target_path; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 316 | offset = 0; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 317 | for (n = 0; n < nmaps; n++) { |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 318 | char *buf; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 319 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 320 | d = XFS_FSB_TO_DADDR(mp, mval[n].br_startblock); |
| 321 | byte_cnt = XFS_FSB_TO_B(mp, mval[n].br_blockcount); |
| 322 | bp = xfs_trans_get_buf(tp, mp->m_ddev_targp, d, |
| 323 | BTOBB(byte_cnt), 0); |
| 324 | if (!bp) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 325 | error = -ENOMEM; |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 326 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 327 | } |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 328 | bp->b_ops = &xfs_symlink_buf_ops; |
| 329 | |
| 330 | byte_cnt = XFS_SYMLINK_BUF_SPACE(mp, byte_cnt); |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 331 | byte_cnt = min(byte_cnt, pathlen); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 332 | |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 333 | buf = bp->b_addr; |
| 334 | buf += xfs_symlink_hdr_set(mp, ip->i_ino, offset, |
| 335 | byte_cnt, bp); |
| 336 | |
| 337 | memcpy(buf, cur_chunk, byte_cnt); |
| 338 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 339 | cur_chunk += byte_cnt; |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 340 | pathlen -= byte_cnt; |
| 341 | offset += byte_cnt; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 342 | |
Dave Chinner | daf7b79 | 2013-09-02 10:32:00 +1000 | [diff] [blame] | 343 | xfs_trans_buf_set_type(tp, bp, XFS_BLFT_SYMLINK_BUF); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 344 | xfs_trans_log_buf(tp, bp, 0, (buf + byte_cnt - 1) - |
| 345 | (char *)bp->b_addr); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 346 | } |
Dave Chinner | 321a958 | 2013-05-27 16:38:20 +1000 | [diff] [blame] | 347 | ASSERT(pathlen == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 348 | } |
| 349 | |
| 350 | /* |
| 351 | * Create the directory entry for the symlink. |
| 352 | */ |
| 353 | error = xfs_dir_createname(tp, dp, link_name, ip->i_ino, |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 354 | &first_block, &dfops, resblks); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 355 | if (error) |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 356 | goto out_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 357 | xfs_trans_ichgtime(tp, dp, XFS_ICHGTIME_MOD | XFS_ICHGTIME_CHG); |
| 358 | xfs_trans_log_inode(tp, dp, XFS_ILOG_CORE); |
| 359 | |
| 360 | /* |
| 361 | * If this is a synchronous mount, make sure that the |
| 362 | * symlink transaction goes to disk before returning to |
| 363 | * the user. |
| 364 | */ |
| 365 | if (mp->m_flags & (XFS_MOUNT_WSYNC|XFS_MOUNT_DIRSYNC)) { |
| 366 | xfs_trans_set_sync(tp); |
| 367 | } |
| 368 | |
Christoph Hellwig | 8ad7c629 | 2017-08-28 10:21:04 -0700 | [diff] [blame] | 369 | error = xfs_defer_finish(&tp, &dfops); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 370 | if (error) |
| 371 | goto out_bmap_cancel; |
| 372 | |
Christoph Hellwig | 7039331 | 2015-06-04 13:48:08 +1000 | [diff] [blame] | 373 | error = xfs_trans_commit(tp); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 374 | if (error) |
| 375 | goto out_release_inode; |
| 376 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 377 | xfs_qm_dqrele(udqp); |
| 378 | xfs_qm_dqrele(gdqp); |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 379 | xfs_qm_dqrele(pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 380 | |
| 381 | *ipp = ip; |
| 382 | return 0; |
| 383 | |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 384 | out_bmap_cancel: |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 385 | xfs_defer_cancel(&dfops); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 386 | out_trans_cancel: |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 387 | xfs_trans_cancel(tp); |
Dave Chinner | 58c9047 | 2015-02-23 22:38:08 +1100 | [diff] [blame] | 388 | out_release_inode: |
| 389 | /* |
| 390 | * Wait until after the current transaction is aborted to finish the |
| 391 | * setup of the inode and release the inode. This prevents recursive |
| 392 | * transactions and deadlocks from xfs_inactive. |
| 393 | */ |
| 394 | if (ip) { |
| 395 | xfs_finish_inode_setup(ip); |
| 396 | IRELE(ip); |
| 397 | } |
| 398 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 399 | xfs_qm_dqrele(udqp); |
| 400 | xfs_qm_dqrele(gdqp); |
Chandra Seetharaman | 92f8ff7 | 2013-07-11 00:00:40 -0500 | [diff] [blame] | 401 | xfs_qm_dqrele(pdqp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 402 | |
| 403 | if (unlock_dp_on_error) |
Christoph Hellwig | 6552321 | 2016-11-30 14:33:25 +1100 | [diff] [blame] | 404 | xfs_iunlock(dp, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 405 | return error; |
| 406 | } |
| 407 | |
| 408 | /* |
| 409 | * Free a symlink that has blocks associated with it. |
| 410 | */ |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 411 | STATIC int |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 412 | xfs_inactive_symlink_rmt( |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 413 | struct xfs_inode *ip) |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 414 | { |
| 415 | xfs_buf_t *bp; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 416 | int done; |
| 417 | int error; |
| 418 | xfs_fsblock_t first_block; |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 419 | struct xfs_defer_ops dfops; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 420 | int i; |
| 421 | xfs_mount_t *mp; |
| 422 | xfs_bmbt_irec_t mval[XFS_SYMLINK_MAPS]; |
| 423 | int nmaps; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 424 | int size; |
| 425 | xfs_trans_t *tp; |
| 426 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 427 | mp = ip->i_mount; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 428 | ASSERT(ip->i_df.if_flags & XFS_IFEXTENTS); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 429 | /* |
| 430 | * We're freeing a symlink that has some |
| 431 | * blocks allocated to it. Free the |
| 432 | * blocks here. We know that we've got |
| 433 | * either 1 or 2 extents and that we can |
| 434 | * free them all in one bunmapi call. |
| 435 | */ |
| 436 | ASSERT(ip->i_d.di_nextents > 0 && ip->i_d.di_nextents <= 2); |
| 437 | |
Christoph Hellwig | 253f491 | 2016-04-06 09:19:55 +1000 | [diff] [blame] | 438 | error = xfs_trans_alloc(mp, &M_RES(mp)->tr_itruncate, 0, 0, 0, &tp); |
| 439 | if (error) |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 440 | return error; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 441 | |
| 442 | xfs_ilock(ip, XFS_ILOCK_EXCL); |
| 443 | xfs_trans_ijoin(tp, ip, 0); |
| 444 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 445 | /* |
| 446 | * Lock the inode, fix the size, and join it to the transaction. |
| 447 | * Hold it so in the normal path, we still have it locked for |
| 448 | * the second transaction. In the error paths we need it |
| 449 | * held so the cancel won't rele it, see below. |
| 450 | */ |
| 451 | size = (int)ip->i_d.di_size; |
| 452 | ip->i_d.di_size = 0; |
| 453 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
| 454 | /* |
| 455 | * Find the block(s) so we can inval and unmap them. |
| 456 | */ |
| 457 | done = 0; |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 458 | xfs_defer_init(&dfops, &first_block); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 459 | nmaps = ARRAY_SIZE(mval); |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 460 | error = xfs_bmapi_read(ip, 0, xfs_symlink_blocks(mp, size), |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 461 | mval, &nmaps, 0); |
| 462 | if (error) |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 463 | goto error_trans_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 464 | /* |
Dave Chinner | f948dd7 | 2013-04-03 16:11:19 +1100 | [diff] [blame] | 465 | * Invalidate the block(s). No validation is done. |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 466 | */ |
| 467 | for (i = 0; i < nmaps; i++) { |
| 468 | bp = xfs_trans_get_buf(tp, mp->m_ddev_targp, |
| 469 | XFS_FSB_TO_DADDR(mp, mval[i].br_startblock), |
| 470 | XFS_FSB_TO_BB(mp, mval[i].br_blockcount), 0); |
| 471 | if (!bp) { |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 472 | error = -ENOMEM; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 473 | goto error_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 474 | } |
| 475 | xfs_trans_binval(tp, bp); |
| 476 | } |
| 477 | /* |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 478 | * Unmap the dead block(s) to the dfops. |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 479 | */ |
Dave Chinner | ab7bb61 | 2015-07-29 11:51:01 +1000 | [diff] [blame] | 480 | error = xfs_bunmapi(tp, ip, 0, size, 0, nmaps, |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 481 | &first_block, &dfops, &done); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 482 | if (error) |
| 483 | goto error_bmap_cancel; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 484 | ASSERT(done); |
| 485 | /* |
| 486 | * Commit the first transaction. This logs the EFI and the inode. |
| 487 | */ |
Christoph Hellwig | 8ad7c629 | 2017-08-28 10:21:04 -0700 | [diff] [blame] | 488 | xfs_defer_ijoin(&dfops, ip); |
| 489 | error = xfs_defer_finish(&tp, &dfops); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 490 | if (error) |
| 491 | goto error_bmap_cancel; |
Dave Chinner | 3565b660 | 2018-05-09 07:49:09 -0700 | [diff] [blame] | 492 | |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 493 | /* |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 494 | * Commit the transaction containing extent freeing and EFDs. |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 495 | */ |
Dave Chinner | 3565b660 | 2018-05-09 07:49:09 -0700 | [diff] [blame] | 496 | xfs_trans_log_inode(tp, ip, XFS_ILOG_CORE); |
Christoph Hellwig | 7039331 | 2015-06-04 13:48:08 +1000 | [diff] [blame] | 497 | error = xfs_trans_commit(tp); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 498 | if (error) { |
| 499 | ASSERT(XFS_FORCED_SHUTDOWN(mp)); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 500 | goto error_unlock; |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 501 | } |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 502 | |
| 503 | /* |
| 504 | * Remove the memory for extent descriptions (just bookkeeping). |
| 505 | */ |
| 506 | if (ip->i_df.if_bytes) |
| 507 | xfs_idata_realloc(ip, -ip->i_df.if_bytes, XFS_DATA_FORK); |
| 508 | ASSERT(ip->i_df.if_bytes == 0); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 509 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 510 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 511 | return 0; |
| 512 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 513 | error_bmap_cancel: |
Darrick J. Wong | 2c3234d | 2016-08-03 11:19:29 +1000 | [diff] [blame] | 514 | xfs_defer_cancel(&dfops); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 515 | error_trans_cancel: |
Christoph Hellwig | 4906e21 | 2015-06-04 13:47:56 +1000 | [diff] [blame] | 516 | xfs_trans_cancel(tp); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 517 | error_unlock: |
| 518 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Dave Chinner | 19de735 | 2013-04-03 16:11:18 +1100 | [diff] [blame] | 519 | return error; |
| 520 | } |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 521 | |
| 522 | /* |
| 523 | * xfs_inactive_symlink - free a symlink |
| 524 | */ |
| 525 | int |
| 526 | xfs_inactive_symlink( |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 527 | struct xfs_inode *ip) |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 528 | { |
| 529 | struct xfs_mount *mp = ip->i_mount; |
| 530 | int pathlen; |
| 531 | |
| 532 | trace_xfs_inactive_symlink(ip); |
| 533 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 534 | if (XFS_FORCED_SHUTDOWN(mp)) |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 535 | return -EIO; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 536 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 537 | xfs_ilock(ip, XFS_ILOCK_EXCL); |
| 538 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 539 | /* |
| 540 | * Zero length symlinks _can_ exist. |
| 541 | */ |
| 542 | pathlen = (int)ip->i_d.di_size; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 543 | if (!pathlen) { |
| 544 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 545 | return 0; |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 546 | } |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 547 | |
Darrick J. Wong | 6eb0b8d | 2017-07-07 08:37:26 -0700 | [diff] [blame] | 548 | if (pathlen < 0 || pathlen > XFS_SYMLINK_MAXLEN) { |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 549 | xfs_alert(mp, "%s: inode (0x%llx) bad symlink length (%d)", |
| 550 | __func__, (unsigned long long)ip->i_ino, pathlen); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 551 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 552 | ASSERT(0); |
Dave Chinner | 2451337 | 2014-06-25 14:58:08 +1000 | [diff] [blame] | 553 | return -EFSCORRUPTED; |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 554 | } |
| 555 | |
| 556 | if (ip->i_df.if_flags & XFS_IFINLINE) { |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 557 | if (ip->i_df.if_bytes > 0) |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 558 | xfs_idata_realloc(ip, -(ip->i_df.if_bytes), |
| 559 | XFS_DATA_FORK); |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 560 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 561 | ASSERT(ip->i_df.if_bytes == 0); |
| 562 | return 0; |
| 563 | } |
| 564 | |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 565 | xfs_iunlock(ip, XFS_ILOCK_EXCL); |
| 566 | |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 567 | /* remove the remote symlink */ |
Brian Foster | 36b21dd | 2013-09-20 11:06:09 -0400 | [diff] [blame] | 568 | return xfs_inactive_symlink_rmt(ip); |
Mark Tinguely | 725eb1e | 2013-06-17 15:35:57 -0500 | [diff] [blame] | 569 | } |