projects
/
platform
/
upstream
/
gst-plugins-good.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
6a16be7
)
rtpvorbisdepay: remove dead code
author
Luis de Bethencourt
<luisbg@osg.samsung.com>
Fri, 1 Apr 2016 11:15:05 +0000
(12:15 +0100)
committer
Luis de Bethencourt
<luisbg@osg.samsung.com>
Fri, 1 Apr 2016 11:15:58 +0000
(12:15 +0100)
payload_buffer hasn't been assigned a value before the jumps to
switch_failed or packet_short. So the value must be NULL. No need
to unmap and unref.
CID #1316476
gst/rtp/gstrtpvorbisdepay.c
patch
|
blob
|
history
diff --git
a/gst/rtp/gstrtpvorbisdepay.c
b/gst/rtp/gstrtpvorbisdepay.c
index
fdd5eb8
..
762750a
100644
(file)
--- a/
gst/rtp/gstrtpvorbisdepay.c
+++ b/
gst/rtp/gstrtpvorbisdepay.c
@@
-612,20
+612,12
@@
switch_failed:
{
GST_ELEMENT_WARNING (rtpvorbisdepay, STREAM, DECODE,
(NULL), ("Could not switch codebooks"));
- if (payload_buffer) {
- gst_buffer_unmap (payload_buffer, &map);
- gst_buffer_unref (payload_buffer);
- }
return NULL;
}
packet_short:
{
GST_ELEMENT_WARNING (rtpvorbisdepay, STREAM, DECODE,
(NULL), ("Packet was too short (%d < 4)", payload_len));
- if (payload_buffer) {
- gst_buffer_unmap (payload_buffer, &map);
- gst_buffer_unref (payload_buffer);
- }
return NULL;
}
ignore_reserved: