diff options
| author | David Howells <dhowells@redhat.com> | 2008-02-07 03:15:26 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-07 11:42:26 -0500 |
| commit | e231c2ee64eb1c5cd3c63c31da9dac7d888dcf7f (patch) | |
| tree | d4b17ef65960594681397a3acac02c2d248200b5 /fs | |
| parent | d1bc8e95445224276d7896b8b08cbb0b28a0ca80 (diff) | |
Convert ERR_PTR(PTR_ERR(p)) instances to ERR_CAST(p)
Convert instances of ERR_PTR(PTR_ERR(p)) to ERR_CAST(p) using:
perl -spi -e 's/ERR_PTR[(]PTR_ERR[(](.*)[)][)]/ERR_CAST(\1)/' `grep -rl 'ERR_PTR[(]*PTR_ERR' fs crypto net security`
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/9p/vfs_inode.c | 2 | ||||
| -rw-r--r-- | fs/affs/namei.c | 2 | ||||
| -rw-r--r-- | fs/afs/dir.c | 4 | ||||
| -rw-r--r-- | fs/afs/security.c | 2 | ||||
| -rw-r--r-- | fs/fat/inode.c | 2 | ||||
| -rw-r--r-- | fs/fuse/dir.c | 6 | ||||
| -rw-r--r-- | fs/gfs2/dir.c | 2 | ||||
| -rw-r--r-- | fs/gfs2/ops_export.c | 2 | ||||
| -rw-r--r-- | fs/gfs2/ops_inode.c | 2 | ||||
| -rw-r--r-- | fs/jffs2/write.c | 4 | ||||
| -rw-r--r-- | fs/nfs/getroot.c | 4 | ||||
| -rw-r--r-- | fs/nfsd/export.c | 4 | ||||
| -rw-r--r-- | fs/quota.c | 4 | ||||
| -rw-r--r-- | fs/reiserfs/inode.c | 2 | ||||
| -rw-r--r-- | fs/reiserfs/xattr.c | 4 | ||||
| -rw-r--r-- | fs/vfat/namei.c | 2 |
16 files changed, 24 insertions, 24 deletions
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c index 5c5137c11484..6a28842052ea 100644 --- a/fs/9p/vfs_inode.c +++ b/fs/9p/vfs_inode.c | |||
| @@ -573,7 +573,7 @@ static struct dentry *v9fs_vfs_lookup(struct inode *dir, struct dentry *dentry, | |||
| 573 | v9ses = v9fs_inode2v9ses(dir); | 573 | v9ses = v9fs_inode2v9ses(dir); |
| 574 | dfid = v9fs_fid_lookup(dentry->d_parent); | 574 | dfid = v9fs_fid_lookup(dentry->d_parent); |
| 575 | if (IS_ERR(dfid)) | 575 | if (IS_ERR(dfid)) |
| 576 | return ERR_PTR(PTR_ERR(dfid)); | 576 | return ERR_CAST(dfid); |
| 577 | 577 | ||
| 578 | name = (char *) dentry->d_name.name; | 578 | name = (char *) dentry->d_name.name; |
| 579 | fid = p9_client_walk(dfid, 1, &name, 1); | 579 | fid = p9_client_walk(dfid, 1, &name, 1); |
diff --git a/fs/affs/namei.c b/fs/affs/namei.c index a42143ca0169..b407e9eea3fb 100644 --- a/fs/affs/namei.c +++ b/fs/affs/namei.c | |||
| @@ -209,7 +209,7 @@ affs_lookup(struct inode *dir, struct dentry *dentry, struct nameidata *nd) | |||
| 209 | bh = affs_find_entry(dir, dentry); | 209 | bh = affs_find_entry(dir, dentry); |
| 210 | affs_unlock_dir(dir); | 210 | affs_unlock_dir(dir); |
| 211 | if (IS_ERR(bh)) { | 211 | if (IS_ERR(bh)) { |
| 212 | return ERR_PTR(PTR_ERR(bh)); | 212 | return ERR_CAST(bh); |
| 213 | } | 213 | } |
| 214 | if (bh) { | 214 | if (bh) { |
| 215 | u32 ino = bh->b_blocknr; | 215 | u32 ino = bh->b_blocknr; |
diff --git a/fs/afs/dir.c b/fs/afs/dir.c index 0cc3597c1197..b58af8f18bc4 100644 --- a/fs/afs/dir.c +++ b/fs/afs/dir.c | |||
| @@ -512,7 +512,7 @@ static struct dentry *afs_lookup(struct inode *dir, struct dentry *dentry, | |||
| 512 | key = afs_request_key(vnode->volume->cell); | 512 | key = afs_request_key(vnode->volume->cell); |
| 513 | if (IS_ERR(key)) { | 513 | if (IS_ERR(key)) { |
| 514 | _leave(" = %ld [key]", PTR_ERR(key)); | 514 | _leave(" = %ld [key]", PTR_ERR(key)); |
| 515 | return ERR_PTR(PTR_ERR(key)); | 515 | return ERR_CAST(key); |
| 516 | } | 516 | } |
| 517 | 517 | ||
| 518 | ret = afs_validate(vnode, key); | 518 | ret = afs_validate(vnode, key); |
| @@ -540,7 +540,7 @@ static struct dentry *afs_lookup(struct inode *dir, struct dentry *dentry, | |||
| 540 | key_put(key); | 540 | key_put(key); |
| 541 | if (IS_ERR(inode)) { | 541 | if (IS_ERR(inode)) { |
| 542 | _leave(" = %ld", PTR_ERR(inode)); | 542 | _leave(" = %ld", PTR_ERR(inode)); |
| 543 | return ERR_PTR(PTR_ERR(inode)); | 543 | return ERR_CAST(inode); |
| 544 | } | 544 | } |
| 545 | 545 | ||
| 546 | dentry->d_op = &afs_fs_dentry_operations; | 546 | dentry->d_op = &afs_fs_dentry_operations; |
diff --git a/fs/afs/security.c b/fs/afs/security.c index 566fe712c682..9446a1fd108a 100644 --- a/fs/afs/security.c +++ b/fs/afs/security.c | |||
| @@ -95,7 +95,7 @@ static struct afs_vnode *afs_get_auth_inode(struct afs_vnode *vnode, | |||
| 95 | auth_inode = afs_iget(vnode->vfs_inode.i_sb, key, | 95 | auth_inode = afs_iget(vnode->vfs_inode.i_sb, key, |
| 96 | &vnode->status.parent, NULL, NULL); | 96 | &vnode->status.parent, NULL, NULL); |
| 97 | if (IS_ERR(auth_inode)) | 97 | if (IS_ERR(auth_inode)) |
| 98 | return ERR_PTR(PTR_ERR(auth_inode)); | 98 | return ERR_CAST(auth_inode); |
| 99 | } | 99 | } |
| 100 | 100 | ||
| 101 | auth_vnode = AFS_FS_I(auth_inode); | 101 | auth_vnode = AFS_FS_I(auth_inode); |
diff --git a/fs/fat/inode.c b/fs/fat/inode.c index 24c0aaa5ae80..3a3d491bbcfe 100644 --- a/fs/fat/inode.c +++ b/fs/fat/inode.c | |||
| @@ -760,7 +760,7 @@ static struct dentry *fat_get_parent(struct dentry *child) | |||
| 760 | inode = fat_build_inode(child->d_sb, de, i_pos); | 760 | inode = fat_build_inode(child->d_sb, de, i_pos); |
| 761 | brelse(bh); | 761 | brelse(bh); |
| 762 | if (IS_ERR(inode)) { | 762 | if (IS_ERR(inode)) { |
| 763 | parent = ERR_PTR(PTR_ERR(inode)); | 763 | parent = ERR_CAST(inode); |
| 764 | goto out; | 764 | goto out; |
| 765 | } | 765 | } |
| 766 | parent = d_alloc_anon(inode); | 766 | parent = d_alloc_anon(inode); |
diff --git a/fs/fuse/dir.c b/fs/fuse/dir.c index f56f91bd38be..7fb514b6d852 100644 --- a/fs/fuse/dir.c +++ b/fs/fuse/dir.c | |||
| @@ -269,12 +269,12 @@ static struct dentry *fuse_lookup(struct inode *dir, struct dentry *entry, | |||
| 269 | 269 | ||
| 270 | req = fuse_get_req(fc); | 270 | req = fuse_get_req(fc); |
| 271 | if (IS_ERR(req)) | 271 | if (IS_ERR(req)) |
| 272 | return ERR_PTR(PTR_ERR(req)); | 272 | return ERR_CAST(req); |
| 273 | 273 | ||
| 274 | forget_req = fuse_get_req(fc); | 274 | forget_req = fuse_get_req(fc); |
| 275 | if (IS_ERR(forget_req)) { | 275 | if (IS_ERR(forget_req)) { |
| 276 | fuse_put_request(fc, req); | 276 | fuse_put_request(fc, req); |
| 277 | return ERR_PTR(PTR_ERR(forget_req)); | 277 | return ERR_CAST(forget_req); |
| 278 | } | 278 | } |
| 279 | 279 | ||
| 280 | attr_version = fuse_get_attr_version(fc); | 280 | attr_version = fuse_get_attr_version(fc); |
| @@ -1006,7 +1006,7 @@ static char *read_link(struct dentry *dentry) | |||
| 1006 | char *link; | 1006 | char *link; |
| 1007 | 1007 | ||
| 1008 | if (IS_ERR(req)) | 1008 | if (IS_ERR(req)) |
| 1009 | return ERR_PTR(PTR_ERR(req)); | 1009 | return ERR_CAST(req); |
| 1010 | 1010 | ||
| 1011 | link = (char *) __get_free_page(GFP_KERNEL); | 1011 | link = (char *) __get_free_page(GFP_KERNEL); |
| 1012 | if (!link) { | 1012 | if (!link) { |
diff --git a/fs/gfs2/dir.c b/fs/gfs2/dir.c index 57e2ed932adc..c34709512b19 100644 --- a/fs/gfs2/dir.c +++ b/fs/gfs2/dir.c | |||
| @@ -1498,7 +1498,7 @@ struct inode *gfs2_dir_search(struct inode *dir, const struct qstr *name) | |||
| 1498 | dent = gfs2_dirent_search(dir, name, gfs2_dirent_find, &bh); | 1498 | dent = gfs2_dirent_search(dir, name, gfs2_dirent_find, &bh); |
| 1499 | if (dent) { | 1499 | if (dent) { |
| 1500 | if (IS_ERR(dent)) | 1500 | if (IS_ERR(dent)) |
| 1501 | return ERR_PTR(PTR_ERR(dent)); | 1501 | return ERR_CAST(dent); |
| 1502 | inode = gfs2_inode_lookup(dir->i_sb, | 1502 | inode = gfs2_inode_lookup(dir->i_sb, |
| 1503 | be16_to_cpu(dent->de_type), | 1503 | be16_to_cpu(dent->de_type), |
| 1504 | be64_to_cpu(dent->de_inum.no_addr), | 1504 | be64_to_cpu(dent->de_inum.no_addr), |
diff --git a/fs/gfs2/ops_export.c b/fs/gfs2/ops_export.c index b9da62348a87..334c7f85351b 100644 --- a/fs/gfs2/ops_export.c +++ b/fs/gfs2/ops_export.c | |||
| @@ -143,7 +143,7 @@ static struct dentry *gfs2_get_parent(struct dentry *child) | |||
| 143 | * have to return that as a(n invalid) pointer to dentry. | 143 | * have to return that as a(n invalid) pointer to dentry. |
| 144 | */ | 144 | */ |
| 145 | if (IS_ERR(inode)) | 145 | if (IS_ERR(inode)) |
| 146 | return ERR_PTR(PTR_ERR(inode)); | 146 | return ERR_CAST(inode); |
| 147 | 147 | ||
| 148 | dentry = d_alloc_anon(inode); | 148 | dentry = d_alloc_anon(inode); |
| 149 | if (!dentry) { | 149 | if (!dentry) { |
diff --git a/fs/gfs2/ops_inode.c b/fs/gfs2/ops_inode.c index 9f71372c1757..e87412902bed 100644 --- a/fs/gfs2/ops_inode.c +++ b/fs/gfs2/ops_inode.c | |||
| @@ -111,7 +111,7 @@ static struct dentry *gfs2_lookup(struct inode *dir, struct dentry *dentry, | |||
| 111 | 111 | ||
| 112 | inode = gfs2_lookupi(dir, &dentry->d_name, 0, nd); | 112 | inode = gfs2_lookupi(dir, &dentry->d_name, 0, nd); |
| 113 | if (inode && IS_ERR(inode)) | 113 | if (inode && IS_ERR(inode)) |
| 114 | return ERR_PTR(PTR_ERR(inode)); | 114 | return ERR_CAST(inode); |
| 115 | 115 | ||
| 116 | if (inode) { | 116 | if (inode) { |
| 117 | struct gfs2_glock *gl = GFS2_I(inode)->i_gl; | 117 | struct gfs2_glock *gl = GFS2_I(inode)->i_gl; |
diff --git a/fs/jffs2/write.c b/fs/jffs2/write.c index 147e2cbee9e4..1b88e6e734ef 100644 --- a/fs/jffs2/write.c +++ b/fs/jffs2/write.c | |||
| @@ -177,7 +177,7 @@ struct jffs2_full_dnode *jffs2_write_dnode(struct jffs2_sb_info *c, struct jffs2 | |||
| 177 | void *hold_err = fn->raw; | 177 | void *hold_err = fn->raw; |
| 178 | /* Release the full_dnode which is now useless, and return */ | 178 | /* Release the full_dnode which is now useless, and return */ |
| 179 | jffs2_free_full_dnode(fn); | 179 | jffs2_free_full_dnode(fn); |
| 180 | return ERR_PTR(PTR_ERR(hold_err)); | 180 | return ERR_CAST(hold_err); |
| 181 | } | 181 | } |
| 182 | fn->ofs = je32_to_cpu(ri->offset); | 182 | fn->ofs = je32_to_cpu(ri->offset); |
| 183 | fn->size = je32_to_cpu(ri->dsize); | 183 | fn->size = je32_to_cpu(ri->dsize); |
| @@ -313,7 +313,7 @@ struct jffs2_full_dirent *jffs2_write_dirent(struct jffs2_sb_info *c, struct jff | |||
| 313 | void *hold_err = fd->raw; | 313 | void *hold_err = fd->raw; |
| 314 | /* Release the full_dirent which is now useless, and return */ | 314 | /* Release the full_dirent which is now useless, and return */ |
| 315 | jffs2_free_full_dirent(fd); | 315 | jffs2_free_full_dirent(fd); |
| 316 | return ERR_PTR(PTR_ERR(hold_err)); | 316 | return ERR_CAST(hold_err); |
| 317 | } | 317 | } |
| 318 | 318 | ||
| 319 | if (retried) { | 319 | if (retried) { |
diff --git a/fs/nfs/getroot.c b/fs/nfs/getroot.c index e6242cdbaf91..fae97196daad 100644 --- a/fs/nfs/getroot.c +++ b/fs/nfs/getroot.c | |||
| @@ -96,7 +96,7 @@ struct dentry *nfs_get_root(struct super_block *sb, struct nfs_fh *mntfh) | |||
| 96 | inode = nfs_fhget(sb, mntfh, fsinfo.fattr); | 96 | inode = nfs_fhget(sb, mntfh, fsinfo.fattr); |
| 97 | if (IS_ERR(inode)) { | 97 | if (IS_ERR(inode)) { |
| 98 | dprintk("nfs_get_root: get root inode failed\n"); | 98 | dprintk("nfs_get_root: get root inode failed\n"); |
| 99 | return ERR_PTR(PTR_ERR(inode)); | 99 | return ERR_CAST(inode); |
| 100 | } | 100 | } |
| 101 | 101 | ||
| 102 | error = nfs_superblock_set_dummy_root(sb, inode); | 102 | error = nfs_superblock_set_dummy_root(sb, inode); |
| @@ -266,7 +266,7 @@ struct dentry *nfs4_get_root(struct super_block *sb, struct nfs_fh *mntfh) | |||
| 266 | inode = nfs_fhget(sb, mntfh, &fattr); | 266 | inode = nfs_fhget(sb, mntfh, &fattr); |
| 267 | if (IS_ERR(inode)) { | 267 | if (IS_ERR(inode)) { |
| 268 | dprintk("nfs_get_root: get root inode failed\n"); | 268 | dprintk("nfs_get_root: get root inode failed\n"); |
| 269 | return ERR_PTR(PTR_ERR(inode)); | 269 | return ERR_CAST(inode); |
| 270 | } | 270 | } |
| 271 | 271 | ||
| 272 | error = nfs_superblock_set_dummy_root(sb, inode); | 272 | error = nfs_superblock_set_dummy_root(sb, inode); |
diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c index 79b4bf812960..346570f6d848 100644 --- a/fs/nfsd/export.c +++ b/fs/nfsd/export.c | |||
| @@ -1218,13 +1218,13 @@ static struct svc_export *exp_find(struct auth_domain *clp, int fsid_type, | |||
| 1218 | struct svc_export *exp; | 1218 | struct svc_export *exp; |
| 1219 | struct svc_expkey *ek = exp_find_key(clp, fsid_type, fsidv, reqp); | 1219 | struct svc_expkey *ek = exp_find_key(clp, fsid_type, fsidv, reqp); |
| 1220 | if (IS_ERR(ek)) | 1220 | if (IS_ERR(ek)) |
| 1221 | return ERR_PTR(PTR_ERR(ek)); | 1221 | return ERR_CAST(ek); |
| 1222 | 1222 | ||
| 1223 | exp = exp_get_by_name(clp, ek->ek_mnt, ek->ek_dentry, reqp); | 1223 | exp = exp_get_by_name(clp, ek->ek_mnt, ek->ek_dentry, reqp); |
| 1224 | cache_put(&ek->h, &svc_expkey_cache); | 1224 | cache_put(&ek->h, &svc_expkey_cache); |
| 1225 | 1225 | ||
| 1226 | if (IS_ERR(exp)) | 1226 | if (IS_ERR(exp)) |
| 1227 | return ERR_PTR(PTR_ERR(exp)); | 1227 | return ERR_CAST(exp); |
| 1228 | return exp; | 1228 | return exp; |
| 1229 | } | 1229 | } |
| 1230 | 1230 | ||
diff --git a/fs/quota.c b/fs/quota.c index 99b24b52bfc8..84f28dd72116 100644 --- a/fs/quota.c +++ b/fs/quota.c | |||
| @@ -341,11 +341,11 @@ static inline struct super_block *quotactl_block(const char __user *special) | |||
| 341 | char *tmp = getname(special); | 341 | char *tmp = getname(special); |
| 342 | 342 | ||
| 343 | if (IS_ERR(tmp)) | 343 | if (IS_ERR(tmp)) |
| 344 | return ERR_PTR(PTR_ERR(tmp)); | 344 | return ERR_CAST(tmp); |
| 345 | bdev = lookup_bdev(tmp); | 345 | bdev = lookup_bdev(tmp); |
| 346 | putname(tmp); | 346 | putname(tmp); |
| 347 | if (IS_ERR(bdev)) | 347 | if (IS_ERR(bdev)) |
| 348 | return ERR_PTR(PTR_ERR(bdev)); | 348 | return ERR_CAST(bdev); |
| 349 | sb = get_super(bdev); | 349 | sb = get_super(bdev); |
| 350 | bdput(bdev); | 350 | bdput(bdev); |
| 351 | if (!sb) | 351 | if (!sb) |
diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c index 195309857e63..57917932212e 100644 --- a/fs/reiserfs/inode.c +++ b/fs/reiserfs/inode.c | |||
| @@ -1536,7 +1536,7 @@ static struct dentry *reiserfs_get_dentry(struct super_block *sb, | |||
| 1536 | if (!inode) | 1536 | if (!inode) |
| 1537 | inode = ERR_PTR(-ESTALE); | 1537 | inode = ERR_PTR(-ESTALE); |
| 1538 | if (IS_ERR(inode)) | 1538 | if (IS_ERR(inode)) |
| 1539 | return ERR_PTR(PTR_ERR(inode)); | 1539 | return ERR_CAST(inode); |
| 1540 | result = d_alloc_anon(inode); | 1540 | result = d_alloc_anon(inode); |
| 1541 | if (!result) { | 1541 | if (!result) { |
| 1542 | iput(inode); | 1542 | iput(inode); |
diff --git a/fs/reiserfs/xattr.c b/fs/reiserfs/xattr.c index a5bd23ce0e46..eba037b3338f 100644 --- a/fs/reiserfs/xattr.c +++ b/fs/reiserfs/xattr.c | |||
| @@ -155,7 +155,7 @@ static struct dentry *get_xa_file_dentry(const struct inode *inode, | |||
| 155 | 155 | ||
| 156 | xadir = open_xa_dir(inode, flags); | 156 | xadir = open_xa_dir(inode, flags); |
| 157 | if (IS_ERR(xadir)) { | 157 | if (IS_ERR(xadir)) { |
| 158 | return ERR_PTR(PTR_ERR(xadir)); | 158 | return ERR_CAST(xadir); |
| 159 | } else if (xadir && !xadir->d_inode) { | 159 | } else if (xadir && !xadir->d_inode) { |
| 160 | dput(xadir); | 160 | dput(xadir); |
| 161 | return ERR_PTR(-ENODATA); | 161 | return ERR_PTR(-ENODATA); |
| @@ -164,7 +164,7 @@ static struct dentry *get_xa_file_dentry(const struct inode *inode, | |||
| 164 | xafile = lookup_one_len(name, xadir, strlen(name)); | 164 | xafile = lookup_one_len(name, xadir, strlen(name)); |
| 165 | if (IS_ERR(xafile)) { | 165 | if (IS_ERR(xafile)) { |
| 166 | dput(xadir); | 166 | dput(xadir); |
| 167 | return ERR_PTR(PTR_ERR(xafile)); | 167 | return ERR_CAST(xafile); |
| 168 | } | 168 | } |
| 169 | 169 | ||
| 170 | if (xafile->d_inode) { /* file exists */ | 170 | if (xafile->d_inode) { /* file exists */ |
diff --git a/fs/vfat/namei.c b/fs/vfat/namei.c index c28add2fbe95..cd450bea9f1a 100644 --- a/fs/vfat/namei.c +++ b/fs/vfat/namei.c | |||
| @@ -705,7 +705,7 @@ static struct dentry *vfat_lookup(struct inode *dir, struct dentry *dentry, | |||
| 705 | brelse(sinfo.bh); | 705 | brelse(sinfo.bh); |
| 706 | if (IS_ERR(inode)) { | 706 | if (IS_ERR(inode)) { |
| 707 | unlock_kernel(); | 707 | unlock_kernel(); |
| 708 | return ERR_PTR(PTR_ERR(inode)); | 708 | return ERR_CAST(inode); |
| 709 | } | 709 | } |
| 710 | alias = d_find_alias(inode); | 710 | alias = d_find_alias(inode); |
| 711 | if (alias) { | 711 | if (alias) { |
