[FFmpeg-cvslog] rtmpproto: Don't include the libavformat version as "clientid"

Martin Storsjö git at videolan.org
Thu Mar 23 17:38:16 EET 2017


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Fri Oct 14 11:30:12 2016 +0300| [9f23f77a532ca9c2b7dc4b5328bc413e4f6f5b56] | committer: Martin Storsjö

rtmpproto: Don't include the libavformat version as "clientid"

When acting as server, the server can include a "clientid" property
in some status messages. But this should be a unique number
identifying the client session, not identifying the server itself.
In practice, omitting it works just as well as including this
incorrect field.

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=9f23f77a532ca9c2b7dc4b5328bc413e4f6f5b56
---

 libavformat/rtmpproto.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c
index 4107b46..0097841 100644
--- a/libavformat/rtmpproto.c
+++ b/libavformat/rtmpproto.c
@@ -1874,9 +1874,6 @@ static int write_status(URLContext *s, RTMPPacket *pkt,
     ff_amf_write_string(&pp, statusmsg);
     ff_amf_write_field_name(&pp, "details");
     ff_amf_write_string(&pp, filename);
-    ff_amf_write_field_name(&pp, "clientid");
-    snprintf(statusmsg, sizeof(statusmsg), "%s", LIBAVFORMAT_IDENT);
-    ff_amf_write_string(&pp, statusmsg);
     ff_amf_write_object_end(&pp);
 
     spkt.size = pp - spkt.data;



More information about the ffmpeg-cvslog mailing list