From: David S. Miller <davem@davemloft.net>
Date: Mon, 18 Apr 2011 00:03:33 +0000 (-0700)
Subject: netfilter: nf_conntrack_standalone: Fix set-but-unused variables.
X-Git-Tag: v3.0-rc1~377^2~347
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=d88d7de09875e643e225a5d0883d18152ce5a89b;p=~shefty%2Frdma-dev.git

netfilter: nf_conntrack_standalone: Fix set-but-unused variables.

The variable 'ret' is set but unused in ct_seq_show().

This was obviously meant to be used to propagate error codes
to the caller, so make it so.

Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/net/netfilter/nf_conntrack_standalone.c b/net/netfilter/nf_conntrack_standalone.c
index 0ae14282588..05e9feb101c 100644
--- a/net/netfilter/nf_conntrack_standalone.c
+++ b/net/netfilter/nf_conntrack_standalone.c
@@ -245,7 +245,7 @@ static int ct_seq_show(struct seq_file *s, void *v)
 	ret = 0;
 release:
 	nf_ct_put(ct);
-	return 0;
+	return ret;
 }
 
 static const struct seq_operations ct_seq_ops = {