diff options
| author | Samuel Pitoiset <samuel.pitoiset@gmail.com> | 2012-08-11 12:41:33 +0200 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2012-08-14 22:31:53 +0300 |
| commit | 8b6a5a7923dc4596091acaa9d8219ade39dd2d61 (patch) | |
| tree | 8882d265450e566f91dd1e45ce0018d5a4e691f3 /libavformat/rtmpproto.c | |
| parent | c0f4cf7798541643f43c6ebbaf3f7780c3d3a8df (diff) | |
rtmp: Do not send _checkbw calls as notifications
The _checkbw calls were changed to use transactionId 0 in commit
82613564 so that servers would not return _result/_error about it.
While this is the strict interpretation of the spec, there are
servers that return _error about it, even if transactionId was 0.
The latest version of EvoStream Media Server (the commercial version
of crtmpserver) behaves properly as described, i.e. returning an
_error normally but not returning anything when using transactionId
0. The latest version of crtmpserver (right now at least) doesn't
behave like this though, it returns an error even if transactionId
was 0.
There are also other servers that return errors even if transactionId
is set to 0. Therefore set a proper transaction id so that the invoke
can be tracked and the error properly ignored instead.
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'libavformat/rtmpproto.c')
| -rw-r--r-- | libavformat/rtmpproto.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index 1db3152e8d..150e45a363 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -626,7 +626,7 @@ static int gen_check_bw(URLContext *s, RTMPContext *rt) p = pkt.data; ff_amf_write_string(&p, "_checkbw"); - ff_amf_write_number(&p, RTMP_NOTIFICATION); + ff_amf_write_number(&p, ++rt->nb_invokes); ff_amf_write_null(&p); return rtmp_send_packet(rt, &pkt, 0); |