tls: rx: fix the false positive warning
[linux-2.6-microblaze.git] / net / tls / tls_sw.c
index fe38b49..17db8c8 100644 (file)
@@ -1283,11 +1283,14 @@ int tls_sw_sendpage(struct sock *sk, struct page *page,
 
 static int
 tls_rx_rec_wait(struct sock *sk, struct sk_psock *psock, bool nonblock,
-               long timeo)
+               bool released)
 {
        struct tls_context *tls_ctx = tls_get_ctx(sk);
        struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);
        DEFINE_WAIT_FUNC(wait, woken_wake_function);
+       long timeo;
+
+       timeo = sock_rcvtimeo(sk, nonblock);
 
        while (!tls_strp_msg_ready(ctx)) {
                if (!sk_psock_queue_empty(psock))
@@ -1297,7 +1300,7 @@ tls_rx_rec_wait(struct sock *sk, struct sk_psock *psock, bool nonblock,
                        return sock_error(sk);
 
                if (!skb_queue_empty(&sk->sk_receive_queue)) {
-                       __strp_unpause(&ctx->strp);
+                       tls_strp_check_rcv(&ctx->strp);
                        if (tls_strp_msg_ready(ctx))
                                break;
                }
@@ -1308,9 +1311,10 @@ tls_rx_rec_wait(struct sock *sk, struct sk_psock *psock, bool nonblock,
                if (sock_flag(sk, SOCK_DONE))
                        return 0;
 
-               if (nonblock || !timeo)
+               if (!timeo)
                        return -EAGAIN;
 
+               released = true;
                add_wait_queue(sk_sleep(sk), &wait);
                sk_set_bit(SOCKWQ_ASYNC_WAITDATA, sk);
                sk_wait_event(sk, &timeo,
@@ -1325,6 +1329,8 @@ tls_rx_rec_wait(struct sock *sk, struct sk_psock *psock, bool nonblock,
                        return sock_intr_errno(timeo);
        }
 
+       tls_strp_msg_load(&ctx->strp, released);
+
        return 1;
 }
 
@@ -1570,7 +1576,7 @@ static int tls_decrypt_sg(struct sock *sk, struct iov_iter *out_iov,
        clear_skb = NULL;
 
        if (unlikely(darg->async)) {
-               err = tls_strp_msg_hold(sk, skb, &ctx->async_hold);
+               err = tls_strp_msg_hold(&ctx->strp, &ctx->async_hold);
                if (err)
                        __skb_queue_tail(&ctx->async_hold, darg->skb);
                return err;
@@ -1631,8 +1637,8 @@ tls_decrypt_sw(struct sock *sk, struct tls_context *tls_ctx,
 }
 
 static int
-tls_decrypt_device(struct sock *sk, struct tls_context *tls_ctx,
-                  struct tls_decrypt_arg *darg)
+tls_decrypt_device(struct sock *sk, struct msghdr *msg,
+                  struct tls_context *tls_ctx, struct tls_decrypt_arg *darg)
 {
        struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);
        struct tls_prot_info *prot = &tls_ctx->prot_info;
@@ -1650,13 +1656,33 @@ tls_decrypt_device(struct sock *sk, struct tls_context *tls_ctx,
        if (pad < 0)
                return pad;
 
-       darg->zc = false;
        darg->async = false;
        darg->skb = tls_strp_msg(ctx);
-       ctx->recv_pkt = NULL;
+       /* ->zc downgrade check, in case TLS 1.3 gets here */
+       darg->zc &= !(prot->version == TLS_1_3_VERSION &&
+                     tls_msg(darg->skb)->control != TLS_RECORD_TYPE_DATA);
 
        rxm = strp_msg(darg->skb);
        rxm->full_len -= pad;
+
+       if (!darg->zc) {
+               /* Non-ZC case needs a real skb */
+               darg->skb = tls_strp_msg_detach(ctx);
+               if (!darg->skb)
+                       return -ENOMEM;
+       } else {
+               unsigned int off, len;
+
+               /* In ZC case nobody cares about the output skb.
+                * Just copy the data here. Note the skb is not fully trimmed.
+                */
+               off = rxm->offset + prot->prepend_size;
+               len = rxm->full_len - prot->overhead_size;
+
+               err = skb_copy_datagram_msg(darg->skb, off, msg, len);
+               if (err)
+                       return err;
+       }
        return 1;
 }
 
@@ -1668,7 +1694,7 @@ static int tls_rx_one_record(struct sock *sk, struct msghdr *msg,
        struct strp_msg *rxm;
        int err;
 
-       err = tls_decrypt_device(sk, tls_ctx, darg);
+       err = tls_decrypt_device(sk, msg, tls_ctx, darg);
        if (!err)
                err = tls_decrypt_sw(sk, tls_ctx, msg, darg);
        if (err < 0)
@@ -1714,9 +1740,7 @@ static int tls_record_content_type(struct msghdr *msg, struct tls_msg *tlm,
 
 static void tls_rx_rec_done(struct tls_sw_context_rx *ctx)
 {
-       consume_skb(ctx->recv_pkt);
-       ctx->recv_pkt = NULL;
-       __strp_unpause(&ctx->strp);
+       tls_strp_msg_done(&ctx->strp);
 }
 
 /* This function traverses the rx_list in tls receive context to copies the
@@ -1803,7 +1827,7 @@ out:
        return copied ? : err;
 }
 
-static void
+static bool
 tls_read_flush_backlog(struct sock *sk, struct tls_prot_info *prot,
                       size_t len_left, size_t decrypted, ssize_t done,
                       size_t *flushed_at)
@@ -1811,18 +1835,18 @@ tls_read_flush_backlog(struct sock *sk, struct tls_prot_info *prot,
        size_t max_rec;
 
        if (len_left <= decrypted)
-               return;
+               return false;
 
        max_rec = prot->overhead_size - prot->tail_size + TLS_MAX_PAYLOAD_SIZE;
        if (done - *flushed_at < SZ_128K && tcp_inq(sk) > max_rec)
-               return;
+               return false;
 
        *flushed_at = done;
-       sk_flush_backlog(sk);
+       return sk_flush_backlog(sk);
 }
 
-static long tls_rx_reader_lock(struct sock *sk, struct tls_sw_context_rx *ctx,
-                              bool nonblock)
+static int tls_rx_reader_lock(struct sock *sk, struct tls_sw_context_rx *ctx,
+                             bool nonblock)
 {
        long timeo;
        int err;
@@ -1853,7 +1877,7 @@ static long tls_rx_reader_lock(struct sock *sk, struct tls_sw_context_rx *ctx,
 
        WRITE_ONCE(ctx->reader_present, 1);
 
-       return timeo;
+       return 0;
 
 err_unlock:
        release_sock(sk);
@@ -1892,10 +1916,10 @@ int tls_sw_recvmsg(struct sock *sk,
        struct tls_msg *tlm;
        ssize_t copied = 0;
        bool async = false;
-       int target, err = 0;
-       long timeo;
+       int target, err;
        bool is_kvec = iov_iter_is_kvec(&msg->msg_iter);
        bool is_peek = flags & MSG_PEEK;
+       bool released = true;
        bool bpf_strp_enabled;
        bool zc_capable;
 
@@ -1903,9 +1927,9 @@ int tls_sw_recvmsg(struct sock *sk,
                return sock_recv_errqueue(sk, msg, len, SOL_IP, IP_RECVERR);
 
        psock = sk_psock_get(sk);
-       timeo = tls_rx_reader_lock(sk, ctx, flags & MSG_DONTWAIT);
-       if (timeo < 0)
-               return timeo;
+       err = tls_rx_reader_lock(sk, ctx, flags & MSG_DONTWAIT);
+       if (err < 0)
+               return err;
        bpf_strp_enabled = sk_psock_strp_enabled(psock);
 
        /* If crypto failed the connection is broken */
@@ -1932,7 +1956,8 @@ int tls_sw_recvmsg(struct sock *sk,
                struct tls_decrypt_arg darg;
                int to_decrypt, chunk;
 
-               err = tls_rx_rec_wait(sk, psock, flags & MSG_DONTWAIT, timeo);
+               err = tls_rx_rec_wait(sk, psock, flags & MSG_DONTWAIT,
+                                     released);
                if (err <= 0) {
                        if (psock) {
                                chunk = sk_msg_recvmsg(sk, psock, msg, len,
@@ -1948,8 +1973,8 @@ int tls_sw_recvmsg(struct sock *sk,
 
                memset(&darg.inargs, 0, sizeof(darg.inargs));
 
-               rxm = strp_msg(ctx->recv_pkt);
-               tlm = tls_msg(ctx->recv_pkt);
+               rxm = strp_msg(tls_strp_msg(ctx));
+               tlm = tls_msg(tls_strp_msg(ctx));
 
                to_decrypt = rxm->full_len - prot->overhead_size;
 
@@ -1988,8 +2013,9 @@ put_on_rx_list_err:
                }
 
                /* periodically flush backlog, and feed strparser */
-               tls_read_flush_backlog(sk, prot, len, to_decrypt,
-                                      decrypted + copied, &flushed_at);
+               released = tls_read_flush_backlog(sk, prot, len, to_decrypt,
+                                                 decrypted + copied,
+                                                 &flushed_at);
 
                /* TLS 1.3 may have updated the length by more than overhead */
                rxm = strp_msg(darg.skb);
@@ -2000,7 +2026,7 @@ put_on_rx_list_err:
                        bool partially_consumed = chunk > len;
                        struct sk_buff *skb = darg.skb;
 
-                       DEBUG_NET_WARN_ON_ONCE(darg.skb == ctx->recv_pkt);
+                       DEBUG_NET_WARN_ON_ONCE(darg.skb == ctx->strp.anchor);
 
                        if (async) {
                                /* TLS 1.2-only, to_decrypt must be text len */
@@ -2014,6 +2040,7 @@ put_on_rx_list:
                        }
 
                        if (bpf_strp_enabled) {
+                               released = true;
                                err = sk_psock_tls_strp_read(psock, skb);
                                if (err != __SK_PASS) {
                                        rxm->offset = rxm->offset + rxm->full_len;
@@ -2106,13 +2133,12 @@ ssize_t tls_sw_splice_read(struct socket *sock,  loff_t *ppos,
        struct tls_msg *tlm;
        struct sk_buff *skb;
        ssize_t copied = 0;
-       int err = 0;
-       long timeo;
        int chunk;
+       int err;
 
-       timeo = tls_rx_reader_lock(sk, ctx, flags & SPLICE_F_NONBLOCK);
-       if (timeo < 0)
-               return timeo;
+       err = tls_rx_reader_lock(sk, ctx, flags & SPLICE_F_NONBLOCK);
+       if (err < 0)
+               return err;
 
        if (!skb_queue_empty(&ctx->rx_list)) {
                skb = __skb_dequeue(&ctx->rx_list);
@@ -2120,7 +2146,7 @@ ssize_t tls_sw_splice_read(struct socket *sock,  loff_t *ppos,
                struct tls_decrypt_arg darg;
 
                err = tls_rx_rec_wait(sk, NULL, flags & SPLICE_F_NONBLOCK,
-                                     timeo);
+                                     true);
                if (err <= 0)
                        goto splice_read_end;
 
@@ -2184,19 +2210,17 @@ bool tls_sw_sock_is_readable(struct sock *sk)
                !skb_queue_empty(&ctx->rx_list);
 }
 
-static int tls_read_size(struct strparser *strp, struct sk_buff *skb)
+int tls_rx_msg_size(struct tls_strparser *strp, struct sk_buff *skb)
 {
        struct tls_context *tls_ctx = tls_get_ctx(strp->sk);
        struct tls_prot_info *prot = &tls_ctx->prot_info;
        char header[TLS_HEADER_SIZE + MAX_IV_SIZE];
-       struct strp_msg *rxm = strp_msg(skb);
-       struct tls_msg *tlm = tls_msg(skb);
        size_t cipher_overhead;
        size_t data_len = 0;
        int ret;
 
        /* Verify that we have a full TLS header, or wait for more data */
-       if (rxm->offset + prot->prepend_size > skb->len)
+       if (strp->stm.offset + prot->prepend_size > skb->len)
                return 0;
 
        /* Sanity-check size of on-stack buffer. */
@@ -2206,11 +2230,11 @@ static int tls_read_size(struct strparser *strp, struct sk_buff *skb)
        }
 
        /* Linearize header to local buffer */
-       ret = skb_copy_bits(skb, rxm->offset, header, prot->prepend_size);
+       ret = skb_copy_bits(skb, strp->stm.offset, header, prot->prepend_size);
        if (ret < 0)
                goto read_failure;
 
-       tlm->control = header[0];
+       strp->mark = header[0];
 
        data_len = ((header[4] & 0xFF) | (header[3] << 8));
 
@@ -2237,7 +2261,7 @@ static int tls_read_size(struct strparser *strp, struct sk_buff *skb)
        }
 
        tls_device_rx_resync_new_rec(strp->sk, data_len + TLS_HEADER_SIZE,
-                                    TCP_SKB_CB(skb)->seq + rxm->offset);
+                                    TCP_SKB_CB(skb)->seq + strp->stm.offset);
        return data_len + TLS_HEADER_SIZE;
 
 read_failure:
@@ -2246,14 +2270,11 @@ read_failure:
        return ret;
 }
 
-static void tls_queue(struct strparser *strp, struct sk_buff *skb)
+void tls_rx_msg_ready(struct tls_strparser *strp)
 {
-       struct tls_context *tls_ctx = tls_get_ctx(strp->sk);
-       struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);
-
-       ctx->recv_pkt = skb;
-       strp_pause(strp);
+       struct tls_sw_context_rx *ctx;
 
+       ctx = container_of(strp, struct tls_sw_context_rx, strp);
        ctx->saved_data_ready(strp->sk);
 }
 
@@ -2263,7 +2284,7 @@ static void tls_data_ready(struct sock *sk)
        struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);
        struct sk_psock *psock;
 
-       strp_data_ready(&ctx->strp);
+       tls_strp_data_ready(&ctx->strp);
 
        psock = sk_psock_get(sk);
        if (psock) {
@@ -2339,13 +2360,11 @@ void tls_sw_release_resources_rx(struct sock *sk)
        kfree(tls_ctx->rx.iv);
 
        if (ctx->aead_recv) {
-               kfree_skb(ctx->recv_pkt);
-               ctx->recv_pkt = NULL;
                __skb_queue_purge(&ctx->rx_list);
                crypto_free_aead(ctx->aead_recv);
-               strp_stop(&ctx->strp);
+               tls_strp_stop(&ctx->strp);
                /* If tls_sw_strparser_arm() was not called (cleanup paths)
-                * we still want to strp_stop(), but sk->sk_data_ready was
+                * we still want to tls_strp_stop(), but sk->sk_data_ready was
                 * never swapped.
                 */
                if (ctx->saved_data_ready) {
@@ -2360,7 +2379,7 @@ void tls_sw_strparser_done(struct tls_context *tls_ctx)
 {
        struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);
 
-       strp_done(&ctx->strp);
+       tls_strp_done(&ctx->strp);
 }
 
 void tls_sw_free_ctx_rx(struct tls_context *tls_ctx)
@@ -2433,8 +2452,6 @@ void tls_sw_strparser_arm(struct sock *sk, struct tls_context *tls_ctx)
        rx_ctx->saved_data_ready = sk->sk_data_ready;
        sk->sk_data_ready = tls_data_ready;
        write_unlock_bh(&sk->sk_callback_lock);
-
-       strp_check_rcv(&rx_ctx->strp);
 }
 
 void tls_update_rx_zc_capable(struct tls_context *tls_ctx)
@@ -2454,7 +2471,6 @@ int tls_set_sw_offload(struct sock *sk, struct tls_context *ctx, int tx)
        struct tls_sw_context_rx *sw_ctx_rx = NULL;
        struct cipher_context *cctx;
        struct crypto_aead **aead;
-       struct strp_callbacks cb;
        u16 nonce_size, tag_size, iv_size, rec_seq_size, salt_size;
        struct crypto_tfm *tfm;
        char *iv, *rec_seq, *key, *salt, *cipher_name;
@@ -2688,12 +2704,7 @@ int tls_set_sw_offload(struct sock *sk, struct tls_context *ctx, int tx)
                        crypto_info->version != TLS_1_3_VERSION &&
                        !!(tfm->__crt_alg->cra_flags & CRYPTO_ALG_ASYNC);
 
-               /* Set up strparser */
-               memset(&cb, 0, sizeof(cb));
-               cb.rcv_msg = tls_queue;
-               cb.parse_msg = tls_read_size;
-
-               strp_init(&sw_ctx_rx->strp, sk, &cb);
+               tls_strp_init(&sw_ctx_rx->strp, sk);
        }
 
        goto out;