ext/speex/gstspeexenc.*: Don't leak adapter.

Original commit message from CVS:
* ext/speex/gstspeexenc.c: (gst_speexenc_class_init),
(gst_speexenc_finalize), (gst_speexenc_sink_setcaps),
(gst_speexenc_chain):
* ext/speex/gstspeexenc.h:
Don't leak adapter.
A push *always* takes ownership of the buffer, even on
errors.
Small cleanups.
This commit is contained in:
Wim Taymans 2006-03-27 14:00:02 +00:00
parent 4c087b76da
commit 6de7310b71
3 changed files with 62 additions and 41 deletions

View file

@ -1,3 +1,14 @@
2006-03-27 Wim Taymans <wim@fluendo.com>
* ext/speex/gstspeexenc.c: (gst_speexenc_class_init),
(gst_speexenc_finalize), (gst_speexenc_sink_setcaps),
(gst_speexenc_chain):
* ext/speex/gstspeexenc.h:
Don't leak adapter.
A push *always* takes ownership of the buffer, even on
errors.
Small cleanups.
2006-03-26 Tim-Philipp Müller <tim at centricular dot net> 2006-03-26 Tim-Philipp Müller <tim at centricular dot net>
* gst/id3demux/gstid3demux.c: (gst_id3demux_add_srcpad): * gst/id3demux/gstid3demux.c: (gst_id3demux_add_srcpad):

View file

@ -98,6 +98,7 @@ gst_speexenc_get_formats (GstPad * pad)
static void gst_speexenc_base_init (gpointer g_class); static void gst_speexenc_base_init (gpointer g_class);
static void gst_speexenc_class_init (GstSpeexEncClass * klass); static void gst_speexenc_class_init (GstSpeexEncClass * klass);
static void gst_speexenc_init (GstSpeexEnc * speexenc); static void gst_speexenc_init (GstSpeexEnc * speexenc);
static void gst_speexenc_finalize (GObject * object);
static gboolean gst_speexenc_sinkevent (GstPad * pad, GstEvent * event); static gboolean gst_speexenc_sinkevent (GstPad * pad, GstEvent * event);
static GstFlowReturn gst_speexenc_chain (GstPad * pad, GstBuffer * buf); static GstFlowReturn gst_speexenc_chain (GstPad * pad, GstBuffer * buf);
@ -232,7 +233,22 @@ gst_speexenc_class_init (GstSpeexEncClass * klass)
parent_class = g_type_class_ref (GST_TYPE_ELEMENT); parent_class = g_type_class_ref (GST_TYPE_ELEMENT);
gstelement_class->change_state = gst_speexenc_change_state; gobject_class->finalize = GST_DEBUG_FUNCPTR (gst_speexenc_finalize);
gstelement_class->change_state =
GST_DEBUG_FUNCPTR (gst_speexenc_change_state);
}
static void
gst_speexenc_finalize (GObject * object)
{
GstSpeexEnc *speexenc;
speexenc = GST_SPEEXENC (object);
g_object_unref (speexenc->adapter);
G_OBJECT_CLASS (parent_class)->finalize (object);
} }
static gboolean static gboolean
@ -241,7 +257,7 @@ gst_speexenc_sink_setcaps (GstPad * pad, GstCaps * caps)
GstSpeexEnc *speexenc; GstSpeexEnc *speexenc;
GstStructure *structure; GstStructure *structure;
speexenc = GST_SPEEXENC (GST_PAD_PARENT (pad)); speexenc = GST_SPEEXENC (gst_pad_get_parent (pad));
speexenc->setup = FALSE; speexenc->setup = FALSE;
structure = gst_caps_get_structure (caps, 0); structure = gst_caps_get_structure (caps, 0);
@ -250,10 +266,9 @@ gst_speexenc_sink_setcaps (GstPad * pad, GstCaps * caps)
gst_speexenc_setup (speexenc); gst_speexenc_setup (speexenc);
if (speexenc->setup) gst_object_unref (speexenc);
return TRUE;
return FALSE; return speexenc->setup;
} }
static gboolean static gboolean
@ -897,13 +912,8 @@ gst_speexenc_chain (GstPad * pad, GstBuffer * buf)
speexenc = GST_SPEEXENC (gst_pad_get_parent (pad)); speexenc = GST_SPEEXENC (gst_pad_get_parent (pad));
if (!speexenc->setup) { if (!speexenc->setup)
gst_buffer_unref (buf); goto not_setup;
GST_ELEMENT_ERROR (speexenc, CORE, NEGOTIATION, (NULL),
("encoder not initialized (input is not audio?)"));
ret = GST_FLOW_UNEXPECTED;
goto error;
}
if (!speexenc->header_sent) { if (!speexenc->header_sent) {
/* Speex streams begin with two headers; the initial header (with /* Speex streams begin with two headers; the initial header (with
@ -943,17 +953,16 @@ gst_speexenc_chain (GstPad * pad, GstBuffer * buf)
ret = gst_speexenc_push_buffer (speexenc, buf1); ret = gst_speexenc_push_buffer (speexenc, buf1);
if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret)) { if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret)) {
gst_buffer_unref (buf1); /* unref buf2 as we are not going to push it anymore */
goto error;
gst_buffer_unref (buf2);
goto done;
} }
ret = gst_speexenc_push_buffer (speexenc, buf2); ret = gst_speexenc_push_buffer (speexenc, buf2);
if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret)) { if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret))
goto done;
gst_buffer_unref (buf2);
goto error;
}
speex_bits_init (&speexenc->bits); speex_bits_init (&speexenc->bits);
speex_bits_reset (&speexenc->bits); speex_bits_reset (&speexenc->bits);
@ -1000,9 +1009,8 @@ gst_speexenc_chain (GstPad * pad, GstBuffer * buf)
GST_BUFFER_OFFSET_NONE, outsize, GST_PAD_CAPS (speexenc->srcpad), GST_BUFFER_OFFSET_NONE, outsize, GST_PAD_CAPS (speexenc->srcpad),
&outbuf); &outbuf);
if ((GST_FLOW_OK != ret)) { if ((GST_FLOW_OK != ret))
goto error; goto done;
}
written = speex_bits_write (&speexenc->bits, written = speex_bits_write (&speexenc->bits,
(gchar *) GST_BUFFER_DATA (outbuf), outsize); (gchar *) GST_BUFFER_DATA (outbuf), outsize);
@ -1010,29 +1018,38 @@ gst_speexenc_chain (GstPad * pad, GstBuffer * buf)
speex_bits_reset (&speexenc->bits); speex_bits_reset (&speexenc->bits);
GST_BUFFER_TIMESTAMP (outbuf) = GST_BUFFER_TIMESTAMP (outbuf) =
(speexenc->frameno * frame_size - gst_util_uint64_scale_int (speexenc->frameno * frame_size -
speexenc->lookahead) * GST_SECOND / speexenc->rate; speexenc->lookahead, GST_SECOND, speexenc->rate);
GST_BUFFER_DURATION (outbuf) = frame_size * GST_SECOND / speexenc->rate; GST_BUFFER_DURATION (outbuf) = gst_util_uint64_scale_int (frame_size,
GST_SECOND, speexenc->rate);
/* set gp time and granulepos; see gst-plugins-base/ext/ogg/README */ /* set gp time and granulepos; see gst-plugins-base/ext/ogg/README */
GST_BUFFER_OFFSET_END (outbuf) = GST_BUFFER_OFFSET_END (outbuf) =
((speexenc->frameno + 1) * frame_size - speexenc->lookahead); ((speexenc->frameno + 1) * frame_size - speexenc->lookahead);
GST_BUFFER_OFFSET (outbuf) = GST_BUFFER_OFFSET (outbuf) =
gst_util_uint64_scale (GST_BUFFER_OFFSET_END (outbuf), GST_SECOND, gst_util_uint64_scale_int (GST_BUFFER_OFFSET_END (outbuf), GST_SECOND,
speexenc->rate); speexenc->rate);
ret = gst_speexenc_push_buffer (speexenc, outbuf); ret = gst_speexenc_push_buffer (speexenc, outbuf);
if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret)) { if ((GST_FLOW_OK != ret) && (GST_FLOW_NOT_LINKED != ret))
gst_buffer_unref (outbuf); goto done;
goto error;
}
} }
} }
error: done:
gst_object_unref (speexenc); gst_object_unref (speexenc);
return ret; return ret;
/* ERRORS */
not_setup:
{
gst_buffer_unref (buf);
GST_ELEMENT_ERROR (speexenc, CORE, NEGOTIATION, (NULL),
("encoder not initialized (input is not audio?)"));
ret = GST_FLOW_NOT_NEGOTIATED;
goto done;
}
} }

View file

@ -28,10 +28,7 @@
#include <speex/speex.h> #include <speex/speex.h>
#include <speex/speex_header.h> #include <speex/speex_header.h>
#ifdef __cplusplus G_BEGIN_DECLS
extern "C" {
#endif /* __cplusplus */
#define GST_TYPE_SPEEXENC \ #define GST_TYPE_SPEEXENC \
(gst_speexenc_get_type()) (gst_speexenc_get_type())
@ -122,10 +119,6 @@ struct _GstSpeexEncClass {
GType gst_speexenc_get_type(void); GType gst_speexenc_get_type(void);
G_END_DECLS
#ifdef __cplusplus
}
#endif /* __cplusplus */
#endif /* __GST_SPEEXENC_H__ */ #endif /* __GST_SPEEXENC_H__ */