From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id D27C74A203 for ; Sun, 23 Jun 2024 22:08:05 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 1314A68D607; Mon, 24 Jun 2024 01:08:02 +0300 (EEST) Received: from btbn.de (btbn.de [144.76.60.213]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 0C91268CD1B for ; Mon, 24 Jun 2024 01:07:56 +0300 (EEST) Received: from [authenticated] by btbn.de (Postfix) with ESMTPSA id 3FB9B27FFD300 for ; Mon, 24 Jun 2024 00:07:55 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rothenpieler.org; s=mail; t=1719180475; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ubOfsM33oAquLFVcbbzp/rAW6To9+9WB0l8dmZ0+kfM=; b=VDFD5ZCZpabxZiB9tadzSX/8M7DmkhsaP/dnmR7l2vnHJA8bY+k17qiZM2dYQ3ICf18Nyg vuvpK+kY8MD/jW1iPfe8HWCUWU/UVDh8u0PJQPwe+wLDk5zTdKwJaA1kFxwmAICY/JPZAr CSv8HttMXRV793JDCMo0/khSPyXBKi8KPNdqKSgyf7QEe9FsZXDruZiOEZ8b7j20G7EMY7 rBJowRtv81xVZ+/Uq4ubFzRyIIyuSURT16VAGirlMyuIuRyUS+EQni/BrMWrEmK9rZiN1x n/yIC6ztJK21jPtf9CuWbB333EMW+ygRF3rsrKB1F/0a6Wac1ClFjtbdwd4Y7g== Message-ID: Date: Mon, 24 Jun 2024 00:07:55 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20240603202811.137136-1-timo@rothenpieler.org> Content-Language: en-US From: Timo Rothenpieler In-Reply-To: <20240603202811.137136-1-timo@rothenpieler.org> Subject: Re: [FFmpeg-devel] [PATCH] avformat/tls_schannel: forward AVIO_FLAG_NONBLOCK to tcp stream X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 03.06.2024 22:28, Timo Rothenpieler wrote: > From: BtbN > > Fixes for example rtmps streaming over schannel. > --- > libavformat/tls_schannel.c | 15 ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-) > > diff --git a/libavformat/tls_schannel.c b/libavformat/tls_schannel.c > index 214a47a218..7265a9794d 100644 > --- a/libavformat/tls_schannel.c > +++ b/libavformat/tls_schannel.c > @@ -113,6 +113,7 @@ static int tls_shutdown_client(URLContext *h) > c->request_flags, 0, 0, NULL, 0, &c->ctxt_handle, > &outbuf_desc, &c->context_flags, &c->ctxt_timestamp); > if (sspi_ret == SEC_E_OK || sspi_ret == SEC_I_CONTEXT_EXPIRED) { > + s->tcp->flags &= ~AVIO_FLAG_NONBLOCK; > ret = ffurl_write(s->tcp, outbuf.pvBuffer, outbuf.cbBuffer); > FreeContextBuffer(outbuf.pvBuffer); > if (ret < 0 || ret != outbuf.cbBuffer) > @@ -316,6 +317,7 @@ static int tls_client_handshake(URLContext *h) > goto fail; > } > > + s->tcp->flags &= ~AVIO_FLAG_NONBLOCK; > ret = ffurl_write(s->tcp, outbuf.pvBuffer, outbuf.cbBuffer); > FreeContextBuffer(outbuf.pvBuffer); > if (ret < 0 || ret != outbuf.cbBuffer) { > @@ -416,11 +418,16 @@ static int tls_read(URLContext *h, uint8_t *buf, int len) > } > } > > + s->tcp->flags &= ~AVIO_FLAG_NONBLOCK; > + s->tcp->flags |= h->flags & AVIO_FLAG_NONBLOCK; > + > ret = ffurl_read(s->tcp, c->enc_buf + c->enc_buf_offset, > c->enc_buf_size - c->enc_buf_offset); > if (ret == AVERROR_EOF) { > c->connection_closed = 1; > ret = 0; > + } else if (ret == AVERROR(EAGAIN)) { > + ret = 0; > } else if (ret < 0) { > av_log(h, AV_LOG_ERROR, "Unable to read from socket\n"); > return ret; > @@ -564,8 +571,14 @@ static int tls_write(URLContext *h, const uint8_t *buf, int len) > sspi_ret = EncryptMessage(&c->ctxt_handle, 0, &outbuf_desc, 0); > if (sspi_ret == SEC_E_OK) { > len = outbuf[0].cbBuffer + outbuf[1].cbBuffer + outbuf[2].cbBuffer; > + > + s->tcp->flags &= ~AVIO_FLAG_NONBLOCK; > + s->tcp->flags |= h->flags & AVIO_FLAG_NONBLOCK; > + > ret = ffurl_write(s->tcp, data, len); > - if (ret < 0 || ret != len) { > + if (ret == AVERROR(EAGAIN)) { > + goto done; > + } else if (ret < 0 || ret != len) { > ret = AVERROR(EIO); > av_log(h, AV_LOG_ERROR, "Writing encrypted data to socket failed\n"); > goto done; will apply soon _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".