Merge branch 'master' into for-upstream
[pandora-kernel.git] / net / dccp / proto.c
index 2cd56df..d0bd348 100644 (file)
@@ -193,7 +193,6 @@ int dccp_init_sock(struct sock *sk, const __u8 ctl_sock_initialized)
 
        dccp_init_xmit_timers(sk);
 
-       INIT_LIST_HEAD(&dp->dccps_featneg);
        /*
         * FIXME: We're hardcoding the CCID, and doing this at this point makes
         * the listening (master) sock get CCID control blocks, which is not
@@ -202,7 +201,7 @@ int dccp_init_sock(struct sock *sk, const __u8 ctl_sock_initialized)
         * setsockopt(CCIDs-I-want/accept). -acme
         */
        if (likely(ctl_sock_initialized)) {
-               int rc = dccp_feat_init(sk);
+               int rc = dccp_feat_init(dmsk);
 
                if (rc)
                        return rc;
@@ -268,7 +267,7 @@ void dccp_destroy_sock(struct sock *sk)
        dp->dccps_hc_rx_ccid = dp->dccps_hc_tx_ccid = NULL;
 
        /* clean up feature negotiation state */
-       dccp_feat_list_purge(&dp->dccps_featneg);
+       dccp_feat_clean(dmsk);
 }
 
 EXPORT_SYMBOL_GPL(dccp_destroy_sock);
@@ -278,9 +277,6 @@ static inline int dccp_listen_start(struct sock *sk, int backlog)
        struct dccp_sock *dp = dccp_sk(sk);
 
        dp->dccps_role = DCCP_ROLE_LISTEN;
-       /* do not start to listen if feature negotiation setup fails */
-       if (dccp_feat_finalise_settings(dp))
-               return -EPROTO;
        return inet_csk_listen_start(sk, backlog);
 }
 
@@ -470,70 +466,42 @@ static int dccp_setsockopt_service(struct sock *sk, const __be32 service,
        return 0;
 }
 
-static int dccp_setsockopt_cscov(struct sock *sk, int cscov, bool rx)
+/* byte 1 is feature.  the rest is the preference list */
+static int dccp_setsockopt_change(struct sock *sk, int type,
+                                 struct dccp_so_feat __user *optval)
 {
-       u8 *list, len;
-       int i, rc;
+       struct dccp_so_feat opt;
+       u8 *val;
+       int rc;
 
-       if (cscov < 0 || cscov > 15)
-               return -EINVAL;
+       if (copy_from_user(&opt, optval, sizeof(opt)))
+               return -EFAULT;
        /*
-        * Populate a list of permissible values, in the range cscov...15. This
-        * is necessary since feature negotiation of single values only works if
-        * both sides incidentally choose the same value. Since the list starts
-        * lowest-value first, negotiation will pick the smallest shared value.
+        * rfc4340: 6.1. Change Options
         */
-       if (cscov == 0)
-               return 0;
-       len = 16 - cscov;
-
-       list = kmalloc(len, GFP_KERNEL);
-       if (list == NULL)
-               return -ENOBUFS;
-
-       for (i = 0; i < len; i++)
-               list[i] = cscov++;
-
-       rc = dccp_feat_register_sp(sk, DCCPF_MIN_CSUM_COVER, rx, list, len);
-
-       if (rc == 0) {
-               if (rx)
-                       dccp_sk(sk)->dccps_pcrlen = cscov;
-               else
-                       dccp_sk(sk)->dccps_pcslen = cscov;
-       }
-       kfree(list);
-       return rc;
-}
-
-static int dccp_setsockopt_ccid(struct sock *sk, int type,
-                               char __user *optval, int optlen)
-{
-       u8 *val;
-       int rc = 0;
-
-       if (optlen < 1 || optlen > DCCP_FEAT_MAX_SP_VALS)
+       if (opt.dccpsf_len < 1)
                return -EINVAL;
 
-       val = kmalloc(optlen, GFP_KERNEL);
-       if (val == NULL)
+       val = kmalloc(opt.dccpsf_len, GFP_KERNEL);
+       if (!val)
                return -ENOMEM;
 
-       if (copy_from_user(val, optval, optlen)) {
-               kfree(val);
-               return -EFAULT;
+       if (copy_from_user(val, opt.dccpsf_val, opt.dccpsf_len)) {
+               rc = -EFAULT;
+               goto out_free_val;
        }
 
-       lock_sock(sk);
-       if (type == DCCP_SOCKOPT_TX_CCID || type == DCCP_SOCKOPT_CCID)
-               rc = dccp_feat_register_sp(sk, DCCPF_CCID, 1, val, optlen);
+       rc = dccp_feat_change(dccp_msk(sk), type, opt.dccpsf_feat,
+                             val, opt.dccpsf_len, GFP_KERNEL);
+       if (rc)
+               goto out_free_val;
 
-       if (!rc && (type == DCCP_SOCKOPT_RX_CCID || type == DCCP_SOCKOPT_CCID))
-               rc = dccp_feat_register_sp(sk, DCCPF_CCID, 0, val, optlen);
-       release_sock(sk);
+out:
+       return rc;
 
+out_free_val:
        kfree(val);
-       return rc;
+       goto out;
 }
 
 static int do_dccp_setsockopt(struct sock *sk, int level, int optname,
@@ -542,21 +510,7 @@ static int do_dccp_setsockopt(struct sock *sk, int level, int optname,
        struct dccp_sock *dp = dccp_sk(sk);
        int val, err = 0;
 
-       switch (optname) {
-       case DCCP_SOCKOPT_PACKET_SIZE:
-               DCCP_WARN("sockopt(PACKET_SIZE) is deprecated: fix your app\n");
-               return 0;
-       case DCCP_SOCKOPT_CHANGE_L:
-       case DCCP_SOCKOPT_CHANGE_R:
-               DCCP_WARN("sockopt(CHANGE_L/R) is deprecated: fix your app\n");
-               return 0;
-       case DCCP_SOCKOPT_CCID:
-       case DCCP_SOCKOPT_RX_CCID:
-       case DCCP_SOCKOPT_TX_CCID:
-               return dccp_setsockopt_ccid(sk, optname, optval, optlen);
-       }
-
-       if (optlen < (int)sizeof(int))
+       if (optlen < sizeof(int))
                return -EINVAL;
 
        if (get_user(val, (int __user *)optval))
@@ -567,24 +521,53 @@ static int do_dccp_setsockopt(struct sock *sk, int level, int optname,
 
        lock_sock(sk);
        switch (optname) {
+       case DCCP_SOCKOPT_PACKET_SIZE:
+               DCCP_WARN("sockopt(PACKET_SIZE) is deprecated: fix your app\n");
+               err = 0;
+               break;
+       case DCCP_SOCKOPT_CHANGE_L:
+               if (optlen != sizeof(struct dccp_so_feat))
+                       err = -EINVAL;
+               else
+                       err = dccp_setsockopt_change(sk, DCCPO_CHANGE_L,
+                                                    (struct dccp_so_feat __user *)
+                                                    optval);
+               break;
+       case DCCP_SOCKOPT_CHANGE_R:
+               if (optlen != sizeof(struct dccp_so_feat))
+                       err = -EINVAL;
+               else
+                       err = dccp_setsockopt_change(sk, DCCPO_CHANGE_R,
+                                                    (struct dccp_so_feat __user *)
+                                                    optval);
+               break;
        case DCCP_SOCKOPT_SERVER_TIMEWAIT:
                if (dp->dccps_role != DCCP_ROLE_SERVER)
                        err = -EOPNOTSUPP;
                else
                        dp->dccps_server_timewait = (val != 0);
                break;
-       case DCCP_SOCKOPT_SEND_CSCOV:
-               err = dccp_setsockopt_cscov(sk, val, false);
+       case DCCP_SOCKOPT_SEND_CSCOV:   /* sender side, RFC 4340, sec. 9.2 */
+               if (val < 0 || val > 15)
+                       err = -EINVAL;
+               else
+                       dp->dccps_pcslen = val;
                break;
-       case DCCP_SOCKOPT_RECV_CSCOV:
-               err = dccp_setsockopt_cscov(sk, val, true);
+       case DCCP_SOCKOPT_RECV_CSCOV:   /* receiver side, RFC 4340 sec. 9.2.1 */
+               if (val < 0 || val > 15)
+                       err = -EINVAL;
+               else {
+                       dp->dccps_pcrlen = val;
+                       /* FIXME: add feature negotiation,
+                        * ChangeL(MinimumChecksumCoverage, val) */
+               }
                break;
        default:
                err = -ENOPROTOOPT;
                break;
        }
-       release_sock(sk);
 
+       release_sock(sk);
        return err;
 }
 
@@ -665,8 +648,6 @@ static int do_dccp_getsockopt(struct sock *sk, int level, int optname,
        case DCCP_SOCKOPT_GET_CUR_MPS:
                val = dp->dccps_mss_cache;
                break;
-       case DCCP_SOCKOPT_AVAILABLE_CCIDS:
-               return ccid_getsockopt_builtin_ccids(sk, len, optval, optlen);
        case DCCP_SOCKOPT_SERVER_TIMEWAIT:
                val = dp->dccps_server_timewait;
                break;