rtsprange: don't leak the range in case of parsing error.
Free the gstRTSPTimeRange if we don't return it. Also simplify gst_rtsp_range_free() as it is valid to pass NULL to g_free().
This commit is contained in:
parent
c074e84360
commit
4e4f922d7a
@ -167,9 +167,10 @@ gst_rtsp_range_parse (const gchar * rangestr, GstRTSPTimeRange ** range)
|
|||||||
} else
|
} else
|
||||||
goto invalid;
|
goto invalid;
|
||||||
|
|
||||||
if (ret == GST_RTSP_OK)
|
if (ret != GST_RTSP_OK)
|
||||||
*range = res;
|
goto invalid;
|
||||||
|
|
||||||
|
*range = res;
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
/* ERRORS */
|
/* ERRORS */
|
||||||
@ -271,8 +272,5 @@ gst_rtsp_range_to_string (const GstRTSPTimeRange * range)
|
|||||||
void
|
void
|
||||||
gst_rtsp_range_free (GstRTSPTimeRange * range)
|
gst_rtsp_range_free (GstRTSPTimeRange * range)
|
||||||
{
|
{
|
||||||
if (range == NULL)
|
|
||||||
return;
|
|
||||||
|
|
||||||
g_free (range);
|
g_free (range);
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user