Module: libav
Branch: master
Commit: 9f23f77a532ca9c2b7dc4b5328bc413e4f6f5b56

Author:    Martin Storsjö <>
Committer: Martin Storsjö <>
Date:      Fri Oct 14 11:30:12 2016 +0300

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ö <>


 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);
     spkt.size = pp -;

libav-commits mailing list

Reply via email to