From patchwork Tue Apr 7 10:07:43 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ross Nicholson X-Patchwork-Id: 18742 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 675214499BC for ; Tue, 7 Apr 2020 13:07:55 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 3EEB168B5C6; Tue, 7 Apr 2020 13:07:55 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wm1-f67.google.com (mail-wm1-f67.google.com [209.85.128.67]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 32F9568B4B2 for ; Tue, 7 Apr 2020 13:07:49 +0300 (EEST) Received: by mail-wm1-f67.google.com with SMTP id g21so214300wmh.0 for ; Tue, 07 Apr 2020 03:07:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=5VK/s+/aR78JZ09zI1+cB+UR922mVZBJh39WxIn+dak=; b=TV+p6FoCgVhL9jJPt+A4SP/cvJEgbg3UN7DeFjMeNp0SG+UA8i8t3Qc5OYoep2nD3S pTs0FYYmm2lGf2e3SwVEge1y4Wra4Idp2q71kGcJwW8s0w2hLlmr9/WEo6aRlLEXLwkX GXUykKpjvtsBPRNQkkRJEQJKAo3oRtGbdwsdqK95UVBqSDSmQKhjqjPY2JPPxL/JXZ5b JEPOzFDWSUcWiyk82iikG74qxOlS6kX7fjXBGLMLp8eqfQOpBWQE4oa25Mq0PBTLBr/D vHWgJpzTJ6lF0coq9Fjk16vSJULmZbHI0dOczZ2luAEwFiRXSVIVICeM23QA7puHQMtg b5Yw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=5VK/s+/aR78JZ09zI1+cB+UR922mVZBJh39WxIn+dak=; b=TqL2gEwhuXD23MgOzRYSp5m8p4pDcNj/3YYcOU5kyXvcCRlNdksLOgEmoelV9NB6b0 BEh3MQpAWoF5IJj6HL45TiMYXEGz19iWmG0D6NJvYqO5HZWmkPxpk0KcdaghDhyBB6so IHYq3qvpoWHOaM1HLA/U5lZ2eBgYZsc9zCcBqNo7EhadkMsBUoAH6hWj/boSnXas+CXN XocmkAUrOo7PdRDVu04oYuGrE/jdqOur//YNVsx8D4dDSi98uVIqVqCIlkinA6ojAHNM /3rT/rQPUjC6zll8G61Cb4eb5hHesvdF34fohPG91C7MUPgSNG2NA2jml89K4o+p+0Um rBuA== X-Gm-Message-State: AGi0PuaziqQSpjJRB3Y3VTQ1bH8jzJ7qaRdeptBkytLpLs7/VjW3pQSx i2qFABmCsZ6cQcTEfo2VOXNkGbab X-Google-Smtp-Source: APiQypLCYPkvE1LHoKBRgJR+KqEatq2/MmaA0jc2WwKwpLST07ADx8IYn6FvA72FoMYya8Y0OUMOWg== X-Received: by 2002:a1c:b743:: with SMTP id h64mr1612698wmf.88.1586254068035; Tue, 07 Apr 2020 03:07:48 -0700 (PDT) Received: from Rosss-MacBook.localdomain ([89.101.86.58]) by smtp.gmail.com with ESMTPSA id k3sm1631537wmf.16.2020.04.07.03.07.47 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 07 Apr 2020 03:07:47 -0700 (PDT) From: phunkyfish To: ffmpeg-devel@ffmpeg.org Date: Tue, 7 Apr 2020 11:07:43 +0100 Message-Id: <20200407100743.54801-1-phunkyfish@gmail.com> X-Mailer: git-send-email 2.24.1 (Apple Git-126) In-Reply-To: <20200406215439.51231-1-phunkyfish@gmail.com> References: <20200406215439.51231-1-phunkyfish@gmail.com> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] avformat/rtp: Pass sources and block filter addresses via sdp file for rtp X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Cc: phunkyfish Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" --- libavformat/rtsp.c | 48 +++++++++++++++++++++++++++++++++++++--------- 1 file changed, 39 insertions(+), 9 deletions(-) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index cd6fc32a29..dad3f7915e 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -21,6 +21,7 @@ #include "libavutil/avassert.h" #include "libavutil/base64.h" +#include "libavutil/bprint.h" #include "libavutil/avstring.h" #include "libavutil/intreadwrite.h" #include "libavutil/mathematics.h" @@ -2447,7 +2448,7 @@ static int rtp_probe(const AVProbeData *p) static int rtp_read_header(AVFormatContext *s) { uint8_t recvbuf[RTP_MAX_PACKET_LENGTH]; - char host[500], sdp[500]; + char host[500], filters_buf[1000]; int ret, port; URLContext* in = NULL; int payload_type; @@ -2456,6 +2457,8 @@ static int rtp_read_header(AVFormatContext *s) AVIOContext pb; socklen_t addrlen = sizeof(addr); RTSPState *rt = s->priv_data; + const char *p; + AVBPrint sdp; if (!ff_network_init()) return AVERROR(EIO); @@ -2513,16 +2516,38 @@ static int rtp_read_header(AVFormatContext *s) av_url_split(NULL, 0, NULL, 0, host, sizeof(host), &port, NULL, 0, s->url); - snprintf(sdp, sizeof(sdp), - "v=0\r\nc=IN IP%d %s\r\nm=%s %d RTP/AVP %d\r\n", - addr.ss_family == AF_INET ? 4 : 6, host, - par->codec_type == AVMEDIA_TYPE_DATA ? "application" : - par->codec_type == AVMEDIA_TYPE_VIDEO ? "video" : "audio", - port, payload_type); - av_log(s, AV_LOG_VERBOSE, "SDP:\n%s\n", sdp); + av_bprint_init(&sdp, 0, AV_BPRINT_SIZE_UNLIMITED); + av_bprintf(&sdp, "v=0\r\nc=IN IP%d %s\r\n", + addr.ss_family == AF_INET ? 4 : 6, host); + + p = strchr(s->url, '?'); + if (p) { + static const char *filters[][2] = {{"sources", "incl"}, {"block", "excl"}, {NULL, NULL}}; + int i; + char *q; + for (i = 0; filters[i][0]; i++) { + if (av_find_info_tag(filters_buf, sizeof(filters_buf), filters[i][0], p)) { + q = filters_buf; + while ((q = strchr(q, ',')) != NULL) + *q = ' '; + av_bprintf(&sdp, "a=source-filter:%s IN IP%d %s %s\r\n", + filters[i][1], + addr.ss_family == AF_INET ? 4 : 6, host, + filters_buf); + } + } + } + + av_bprintf(&sdp, "m=%s %d RTP/AVP %d\r\n", + par->codec_type == AVMEDIA_TYPE_DATA ? "application" : + par->codec_type == AVMEDIA_TYPE_VIDEO ? "video" : "audio", + port, payload_type); + av_log(s, AV_LOG_VERBOSE, "SDP:\n%s\n", sdp.str); + if (av_bprint_is_complete(&sdp)) + goto fail_nobuf; avcodec_parameters_free(&par); - ffio_init_context(&pb, sdp, strlen(sdp), 0, NULL, NULL, NULL, NULL); + ffio_init_context(&pb, sdp.str, strlen(sdp.str), 0, NULL, NULL, NULL, NULL); s->pb = &pb; /* sdp_read_header initializes this again */ @@ -2532,9 +2557,14 @@ static int rtp_read_header(AVFormatContext *s) ret = sdp_read_header(s); s->pb = NULL; + av_bprint_finalize(&sdp, NULL); return ret; +fail_nobuf: + ret = AVERROR(ENOMEM); + av_log(s, AV_LOG_ERROR, "rtp_read_header(): not enough buffer space for sdp-headers\n"); fail: + av_bprint_finalize(&sdp, NULL); avcodec_parameters_free(&par); if (in) ffurl_close(in);