rtmpproto: Don't include the libavformat version as "clientid"
authorMartin Storsjö <martin@martin.st>
Fri, 14 Oct 2016 08:30:12 +0000 (11:30 +0300)
committerMartin Storsjö <martin@martin.st>
Fri, 14 Oct 2016 20:11:17 +0000 (23:11 +0300)
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@martin.st>
libavformat/rtmpproto.c

index 4107b46..0097841 100644 (file)
@@ -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;