summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAl Viro <viro@ZenIV.linux.org.uk>2011-06-05 00:54:03 +0000
committerDavid S. Miller <davem@davemloft.net>2011-06-05 14:11:09 -0700
commitb8f07a063163f8216cd891c5b007e839a56b6d93 (patch)
tree83cd31339371ab707d29e1275633baf4d3db2a65
parentc316e6a3084cef1a5857cd66bb5429c969f06c93 (diff)
downloadop-kernel-dev-b8f07a063163f8216cd891c5b007e839a56b6d93.zip
op-kernel-dev-b8f07a063163f8216cd891c5b007e839a56b6d93.tar.gz
fix return values of l2tp_dfs_seq_open()
More fallout from struct net lifetime rules review: PTR_ERR() is *already* negative and failing ->open() should return negatives on failure. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: James Chapman <jchapman@katalix.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/l2tp/l2tp_debugfs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/l2tp/l2tp_debugfs.c b/net/l2tp/l2tp_debugfs.c
index b8dbae8..7613013 100644
--- a/net/l2tp/l2tp_debugfs.c
+++ b/net/l2tp/l2tp_debugfs.c
@@ -258,7 +258,7 @@ static int l2tp_dfs_seq_open(struct inode *inode, struct file *file)
*/
pd->net = get_net_ns_by_pid(current->pid);
if (IS_ERR(pd->net)) {
- rc = -PTR_ERR(pd->net);
+ rc = PTR_ERR(pd->net);
goto err_free_pd;
}
OpenPOWER on IntegriCloud