diff options
author | Jan Alexander Steffens (heftig) <jsteffens@make.tv> | 2019-11-27 15:36:07 +0100 |
---|---|---|
committer | Jan Alexander Steffens (heftig) <jsteffens@make.tv> | 2019-12-03 14:11:46 +0100 |
commit | f5b068b26ce71a7025bc23d589dc5a14e6461317 (patch) | |
tree | d81f3d64c0cd4c0c14777f165012b9d3978bea6c /gst/rtmp2 | |
parent | a9a2c3d67a84b7686977bf7ffaf85303c97534e0 (diff) | |
download | gstreamer-plugins-bad-f5b068b26ce71a7025bc23d589dc5a14e6461317.tar.gz |
rtmp2: Minor changes
- Remove an unneeded initialization to zero from AmfParser
- Add missing initialization to gst_amf_serialize_command_valist
- Add a g_return_if_fail to gst_rtmp_connection_request_window_size
https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/merge_requests/878
Diffstat (limited to 'gst/rtmp2')
-rw-r--r-- | gst/rtmp2/rtmp/amf.c | 3 | ||||
-rw-r--r-- | gst/rtmp2/rtmp/rtmpconnection.c | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/gst/rtmp2/rtmp/amf.c b/gst/rtmp2/rtmp/amf.c index a6f5b3be0..895570f49 100644 --- a/gst/rtmp2/rtmp/amf.c +++ b/gst/rtmp2/rtmp/amf.c @@ -893,7 +893,6 @@ gst_amf_parse_command (const guint8 * data, gsize size, AmfParser parser = { .data = data, .size = size, - .recursion_depth = 0, }; GstAmfNode *node1 = NULL, *node2 = NULL; GPtrArray *args = NULL; @@ -1119,6 +1118,8 @@ gst_amf_serialize_command_valist (gdouble transaction_id, g_return_val_if_fail (command_name, NULL); g_return_val_if_fail (argument, NULL); + init_static (); + GST_LOG ("Serializing command '%s', transid %.0f", command_name, transaction_id); diff --git a/gst/rtmp2/rtmp/rtmpconnection.c b/gst/rtmp2/rtmp/rtmpconnection.c index 8c14516b0..d895f8e5c 100644 --- a/gst/rtmp2/rtmp/rtmpconnection.c +++ b/gst/rtmp2/rtmp/rtmpconnection.c @@ -989,6 +989,8 @@ gst_rtmp_connection_request_window_size (GstRtmpConnection * connection, .param = window_ack_size, }; + g_return_if_fail (GST_IS_RTMP_CONNECTION (connection)); + if (connection->out_window_ack_size == window_ack_size) return; |