# HG changeset patch # User mstorsjo # Date 1268295858 0 # Node ID 53c5a5d40ee72e52a89b1a218a1957f15cf99f28 # Parent 1a3c4f74f17d99263e40805f36db0efd138ea949 RTSP muxer: Create the SDP with the numerical IP of the peer instead of using the original host name diff -r 1a3c4f74f17d -r 53c5a5d40ee7 rtsp.c --- a/rtsp.c Thu Mar 11 07:18:00 2010 +0000 +++ b/rtsp.c Thu Mar 11 08:24:18 2010 +0000 @@ -1328,13 +1328,14 @@ return 0; } -static int rtsp_setup_output_streams(AVFormatContext *s) +static int rtsp_setup_output_streams(AVFormatContext *s, const char *addr) { RTSPState *rt = s->priv_data; RTSPMessageHeader reply1, *reply = &reply1; char cmd[1024]; int i; char *sdp; + AVFormatContext sdp_ctx, *ctx_array[1]; /* Announce the stream */ snprintf(cmd, sizeof(cmd), @@ -1344,7 +1345,23 @@ sdp = av_mallocz(8192); if (sdp == NULL) return AVERROR(ENOMEM); - if (avf_sdp_create(&s, 1, sdp, 8192)) { + /* We create the SDP based on the RTSP AVFormatContext where we + * aren't allowed to change the filename field. (We create the SDP + * based on the RTSP context since the contexts for the RTP streams + * don't exist yet.) In order to specify a custom URL with the actual + * peer IP instead of the originally specified hostname, we create + * a temporary copy of the AVFormatContext, where the custom URL is set. + * + * FIXME: Create the SDP without copying the AVFormatContext. + * This either requires setting up the RTP stream AVFormatContexts + * already here (complicating things immensely) or getting a more + * flexible SDP creation interface. + */ + sdp_ctx = *s; + ff_url_join(sdp_ctx.filename, sizeof(sdp_ctx.filename), + "rtsp", NULL, addr, -1, NULL); + ctx_array[0] = &sdp_ctx; + if (avf_sdp_create(ctx_array, 1, sdp, 8192)) { av_free(sdp); return AVERROR_INVALIDDATA; } @@ -1507,7 +1524,7 @@ if (s->iformat) err = rtsp_setup_input_streams(s); else - err = rtsp_setup_output_streams(s); + err = rtsp_setup_output_streams(s, host); if (err) goto fail;