From d48e57538dc2b1b9eb89aa66cf1c924e45cc286f Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Mon, 26 Jan 2015 12:49:45 +0000 Subject: [PATCH] aiffparse: fix which stop variable is used in assignment Assignment is done to variable segment.stop when the intention was to assign to local variable stop. Instead of overwriting it, the value is now clamped and segment.stop is set to it soon after. CID #1265772 --- gst/aiff/aiffparse.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gst/aiff/aiffparse.c b/gst/aiff/aiffparse.c index da45ce5a23..6bae23a608 100644 --- a/gst/aiff/aiffparse.c +++ b/gst/aiff/aiffparse.c @@ -1818,8 +1818,8 @@ gst_aiff_parse_sink_event (GstPad * pad, GstObject * parent, GstEvent * event) } if (stop > 0) { end_offset = stop; - segment.stop -= aiff->datastart; - segment.stop = MAX (stop, 0); + stop -= aiff->datastart; + stop = MAX (stop, 0); } if (aiff->state == AIFF_PARSE_DATA && aiff->segment.format == GST_FORMAT_TIME) {