* [PATCH]: Fix ipcomp warnings...
@ 2008-07-27 11:00 David Miller
2008-07-27 11:39 ` Herbert Xu
0 siblings, 1 reply; 2+ messages in thread
From: David Miller @ 2008-07-27 11:00 UTC (permalink / raw)
To: herbert; +Cc: netdev
Herbert just FYI, I checked the following into net-2.6
ipcomp: Fix warnings after ipcomp consolidation.
net/ipv4/ipcomp.c: In function ‘ipcomp4_init_state’:
net/ipv4/ipcomp.c:109: warning: unused variable ‘calg_desc’
net/ipv4/ipcomp.c:108: warning: unused variable ‘ipcd’
net/ipv4/ipcomp.c:107: warning: ‘err’ may be used uninitialized in this function
net/ipv6/ipcomp6.c: In function ‘ipcomp6_init_state’:
net/ipv6/ipcomp6.c:139: warning: unused variable ‘calg_desc’
net/ipv6/ipcomp6.c:138: warning: unused variable ‘ipcd’
net/ipv6/ipcomp6.c:137: warning: ‘err’ may be used uninitialized in this function
Signed-off-by: David S. Miller <davem@davemloft.net>
---
net/ipv4/ipcomp.c | 4 +---
net/ipv6/ipcomp6.c | 4 +---
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/net/ipv4/ipcomp.c b/net/ipv4/ipcomp.c
index a42b64d..38ccb6d 100644
--- a/net/ipv4/ipcomp.c
+++ b/net/ipv4/ipcomp.c
@@ -104,9 +104,7 @@ out:
static int ipcomp4_init_state(struct xfrm_state *x)
{
- int err;
- struct ipcomp_data *ipcd;
- struct xfrm_algo_desc *calg_desc;
+ int err = -EINVAL;
x->props.header_len = 0;
switch (x->props.mode) {
diff --git a/net/ipv6/ipcomp6.c b/net/ipv6/ipcomp6.c
index 0cfcea4..4545e43 100644
--- a/net/ipv6/ipcomp6.c
+++ b/net/ipv6/ipcomp6.c
@@ -134,9 +134,7 @@ out:
static int ipcomp6_init_state(struct xfrm_state *x)
{
- int err;
- struct ipcomp_data *ipcd;
- struct xfrm_algo_desc *calg_desc;
+ int err = -EINVAL;
x->props.header_len = 0;
switch (x->props.mode) {
--
1.6.0.rc0.14.g95f8
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH]: Fix ipcomp warnings...
2008-07-27 11:00 [PATCH]: Fix ipcomp warnings David Miller
@ 2008-07-27 11:39 ` Herbert Xu
0 siblings, 0 replies; 2+ messages in thread
From: Herbert Xu @ 2008-07-27 11:39 UTC (permalink / raw)
To: David Miller; +Cc: netdev
On Sun, Jul 27, 2008 at 04:00:36AM -0700, David Miller wrote:
>
> Herbert just FYI, I checked the following into net-2.6
Thanks Dave! I think when I tested this somehow the kernel decided
to do a total rebuild so the warnings slipped through.
Cheers,
--
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-07-27 11:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-07-27 11:00 [PATCH]: Fix ipcomp warnings David Miller
2008-07-27 11:39 ` Herbert Xu
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox