Commit 5cf368b3 authored by Arun Raghavan's avatar Arun Raghavan

dcaparse: Don't unmap buffer before accessing data from it

The previous patch added a check for a substream header after
gst_buffer_unmap(), which is incorrect.
parent f7854411
...@@ -446,9 +446,6 @@ gst_dca_parse_handle_frame (GstBaseParse * parse, ...@@ -446,9 +446,6 @@ gst_dca_parse_handle_frame (GstBaseParse * parse,
gst_base_parse_set_frame_rate (parse, rate, block_size, 0, 0); gst_base_parse_set_frame_rate (parse, rate, block_size, 0, 0);
} }
cleanup:
gst_buffer_unmap (buf, &map);
/* it is possible that DTS HD substream after DTS core */ /* it is possible that DTS HD substream after DTS core */
if (parse->flags & GST_BASE_PARSE_FLAG_DRAINING || map.size >= size + 9) { if (parse->flags & GST_BASE_PARSE_FLAG_DRAINING || map.size >= size + 9) {
extra_size = 0; extra_size = 0;
...@@ -481,6 +478,9 @@ cleanup: ...@@ -481,6 +478,9 @@ cleanup:
ret = GST_FLOW_OK; ret = GST_FLOW_OK;
} }
cleanup:
gst_buffer_unmap (buf, &map);
return ret; return ret;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment