Comcast / gots

MPEG Transport Stream handling in Go
Other
308 stars 88 forks source link

Handle case where PAT section length is too small #129

Closed eric closed 5 years ago

eric commented 5 years ago

There are cases where the subscription math causes the uint16 to wrap around. Casting it to an int prevents this from happening

igilham commented 5 years ago

Is it worth adding a test case that this change fixes?

eric commented 5 years ago

I'm not sure — it's hard to create small test cases for these sorts of situations.