diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2012-08-25 15:57:04 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-09-10 17:46:17 -0400 |
commit | 92566e287d5211774f9466e35d7b7240532521e6 (patch) | |
tree | 121b548cc72d3e7c9a1300ec9545c9168df0eaf1 /fs | |
parent | eccf50c129686de11358093839749c83f6cae5db (diff) |
fs/nfsd/nfs4idmap.c: adjust inconsistent IS_ERR and PTR_ERR
Change the call to PTR_ERR to access the value just tested by IS_ERR.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression e,e1;
@@
(
if (IS_ERR(e)) { ... PTR_ERR(e) ... }
|
if (IS_ERR(e=e1)) { ... PTR_ERR(e) ... }
|
*if (IS_ERR(e))
{ ...
* PTR_ERR(e1)
... }
)
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/nfsd/nfs4idmap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c index fdc91a6fc9c4..11df4aca89ba 100644 --- a/fs/nfsd/nfs4idmap.c +++ b/fs/nfsd/nfs4idmap.c | |||
@@ -478,7 +478,7 @@ nfsd_idmap_init(struct net *net) | |||
478 | goto destroy_idtoname_cache; | 478 | goto destroy_idtoname_cache; |
479 | nn->nametoid_cache = cache_create_net(&nametoid_cache_template, net); | 479 | nn->nametoid_cache = cache_create_net(&nametoid_cache_template, net); |
480 | if (IS_ERR(nn->nametoid_cache)) { | 480 | if (IS_ERR(nn->nametoid_cache)) { |
481 | rv = PTR_ERR(nn->idtoname_cache); | 481 | rv = PTR_ERR(nn->nametoid_cache); |
482 | goto unregister_idtoname_cache; | 482 | goto unregister_idtoname_cache; |
483 | } | 483 | } |
484 | rv = cache_register_net(nn->nametoid_cache, net); | 484 | rv = cache_register_net(nn->nametoid_cache, net); |