mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2025-02-02 04:22:27 +00:00
rtmp2/client: Make sure 'code' is not NULL
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1403>
This commit is contained in:
parent
a100f36b69
commit
2037f1ed0a
1 changed files with 4 additions and 4 deletions
|
@ -722,15 +722,15 @@ send_connect_done (const gchar * command_name, GPtrArray * args,
|
||||||
optional_args = g_ptr_array_index (args, 1);
|
optional_args = g_ptr_array_index (args, 1);
|
||||||
|
|
||||||
node = gst_amf_node_get_field (optional_args, "code");
|
node = gst_amf_node_get_field (optional_args, "code");
|
||||||
if (!node) {
|
code = node ? gst_amf_node_peek_string (node, NULL) : NULL;
|
||||||
|
if (!code) {
|
||||||
g_task_return_new_error (task, G_IO_ERROR, G_IO_ERROR_FAILED,
|
g_task_return_new_error (task, G_IO_ERROR, G_IO_ERROR_FAILED,
|
||||||
"result code missing from connect cmd result");
|
"result code missing from connect cmd result");
|
||||||
g_object_unref (task);
|
g_object_unref (task);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
code = gst_amf_node_peek_string (node, NULL);
|
GST_INFO ("connect result: %s", code);
|
||||||
GST_INFO ("connect result: %s", GST_STR_NULL (code));
|
|
||||||
|
|
||||||
if (g_str_equal (code, "NetConnection.Connect.Success")) {
|
if (g_str_equal (code, "NetConnection.Connect.Success")) {
|
||||||
node = gst_amf_node_get_field (optional_args, "secureToken");
|
node = gst_amf_node_get_field (optional_args, "secureToken");
|
||||||
|
@ -859,7 +859,7 @@ send_connect_done (const gchar * command_name, GPtrArray * args,
|
||||||
}
|
}
|
||||||
|
|
||||||
g_task_return_new_error (task, G_IO_ERROR, G_IO_ERROR_FAILED,
|
g_task_return_new_error (task, G_IO_ERROR, G_IO_ERROR_FAILED,
|
||||||
"unhandled connect result code: %s", GST_STR_NULL (code));
|
"unhandled connect result code: %s", code);
|
||||||
g_object_unref (task);
|
g_object_unref (task);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue