Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=1dff92e09eb3adfcf3d3dffd0507b850f911f280
Commit:     1dff92e09eb3adfcf3d3dffd0507b850f911f280
Parent:     e6a5fdf56e3a5fc179cd8c8c19081a9a11882b0c
Author:     Adrian Bunk <[EMAIL PROTECTED]>
AuthorDate: Mon Oct 15 01:37:55 2007 -0700
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Mon Oct 15 12:26:32 2007 -0700

    [IPV6] __inet6_csk_dst_store(): fix check-after-use
    
    The Coverity checker spotted that we have already oops'ed if "dst" was
    NULL.
    
    Since "dst" being NULL doesn't seem to be possible at this point this
    patch removes the NULL check.
    
    Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
    Acked-by: Masahide NAKAMURA <[EMAIL PROTECTED]>
    Acked-by: Noriaki TAKAMIYA <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 net/ipv6/inet6_connection_sock.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/ipv6/inet6_connection_sock.c b/net/ipv6/inet6_connection_sock.c
index 25b9317..78de42a 100644
--- a/net/ipv6/inet6_connection_sock.c
+++ b/net/ipv6/inet6_connection_sock.c
@@ -146,7 +146,7 @@ void __inet6_csk_dst_store(struct sock *sk, struct 
dst_entry *dst,
        __ip6_dst_store(sk, dst, daddr, saddr);
 
 #ifdef CONFIG_XFRM
-       if (dst) {
+       {
                struct rt6_info *rt = (struct rt6_info  *)dst;
                rt->rt6i_flow_cache_genid = atomic_read(&flow_cache_genid);
        }
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to