net: sctp: minor: remove variable in sctp_init_sock
authorDaniel Borkmann <dborkman@redhat.com>
Fri, 14 Jun 2013 16:24:06 +0000 (18:24 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 14 Jun 2013 22:38:36 +0000 (15:38 -0700)
It's only used at this one time, so we could remove it as well.
This is valid and also makes it more explicit/obvious that in case
of error the sp->ep is NULL here, i.e. for the sctp_destroy_sock()
check that was recently added.

Signed-off-by: Daniel Borkmann <dborkman@redhat.com>
Acked-by: Vlad Yasevich <vyasevich@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/socket.c

index f631c5ff4dbfd84426d140920b15650972f34646..510dc79a32a1c4dddc985d6568bed2e4f1340157 100644 (file)
@@ -3862,7 +3862,6 @@ out:
 SCTP_STATIC int sctp_init_sock(struct sock *sk)
 {
        struct net *net = sock_net(sk);
-       struct sctp_endpoint *ep;
        struct sctp_sock *sp;
 
        SCTP_DEBUG_PRINTK("sctp_init_sock(sk: %p)\n", sk);
@@ -3971,11 +3970,10 @@ SCTP_STATIC int sctp_init_sock(struct sock *sk)
         * change the data structure relationships, this may still
         * be useful for storing pre-connect address information.
         */
-       ep = sctp_endpoint_new(sk, GFP_KERNEL);
-       if (!ep)
+       sp->ep = sctp_endpoint_new(sk, GFP_KERNEL);
+       if (!sp->ep)
                return -ENOMEM;
 
-       sp->ep = ep;
        sp->hmac = NULL;
 
        SCTP_DBG_OBJCNT_INC(sock);