[IPV4/6]: Fix UFO error propagation

When ufo_append_data fails err is uninitialized, but returned back.
Strangely gcc doesn't notice it.

Coverity #901 and #902

Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]>

---
commit 3a83b01bbd9e0a2becfc427f7866dd2f20f75494
tree 421450cdd094d6490cfe4b561610c25ff5ca0e22
parent cbb18ea472603ef2e7fcc0dd21490c22a9c01335
author Patrick McHardy <[EMAIL PROTECTED]> Mon, 13 Mar 2006 00:09:33 +0100
committer Patrick McHardy <[EMAIL PROTECTED]> Mon, 13 Mar 2006 00:09:33 +0100

 net/ipv4/ip_output.c  |    5 +++--
 net/ipv6/ip6_output.c |    5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c
index 57d290d..46b0771 100644
--- a/net/ipv4/ip_output.c
+++ b/net/ipv4/ip_output.c
@@ -847,8 +847,9 @@ int ip_append_data(struct sock *sk,
        if (((length > mtu) && (sk->sk_protocol == IPPROTO_UDP)) &&
                        (rt->u.dst.dev->features & NETIF_F_UFO)) {
 
-               if(ip_ufo_append_data(sk, getfrag, from, length, hh_len,
-                              fragheaderlen, transhdrlen, mtu, flags))
+               if ((err = ip_ufo_append_data(sk, getfrag, from, length, hh_len,
+                                             fragheaderlen, transhdrlen, mtu,
+                                             flags)))
                        goto error;
 
                return 0;
diff --git a/net/ipv6/ip6_output.c b/net/ipv6/ip6_output.c
index f999edd..5fe62d0 100644
--- a/net/ipv6/ip6_output.c
+++ b/net/ipv6/ip6_output.c
@@ -944,8 +944,9 @@ int ip6_append_data(struct sock *sk, int
        if (((length > mtu) && (sk->sk_protocol == IPPROTO_UDP)) &&
            (rt->u.dst.dev->features & NETIF_F_UFO)) {
 
-               if(ip6_ufo_append_data(sk, getfrag, from, length, hh_len,
-                               fragheaderlen, transhdrlen, mtu, flags))
+               if ((err = ip6_ufo_append_data(sk, getfrag, from, length, 
hh_len,
+                                              fragheaderlen, transhdrlen, mtu,
+                                              flags)))
                        goto error;
 
                return 0;
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to