[Bug 750847] dashdemux: variables containing time information should be guint64 not gint64

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=750847

--- Comment #20 from Florin Apostol <florin.apostol at oregan.net> ---
(In reply to Vincent Penquerc'h from comment #19)
> OK, I will make this accept leading whitespace them.
> 
> As for 33xx, we specifically want that to allow data after it (ie, Y2M3).
> But in the 333333333333333 case, the extra digits are eaten by the %u
> parser, causing overflow, rather than the parsing stopping (which would be
> weird too I guess).

We want to allow data after a number only in special cases (e.g.
P0Y1M2DT12H10M20.5S) and in such cases the parsing is more elaborate, verifying
the presence of expected markers.
When I mentioned 33xx I was referring to fields that contain numbers only (eg
presentationTimeOffset) for which no extra validation is done if the field
starts with a number.
Bottom line, I believe we should check all the bytes present in a field and
issue a warning if anything extra (not mentioned in the standard) is found. If
the extra data is spaces, we accept the value. If the extra data is something
else, we should use the default value for the field.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.