<div dir="ltr">Oops pressed send too soon.<br><div><div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 23, 2013 at 8:22 PM, Jean-Paul Saman <span dir="ltr"><<a href="mailto:jpsaman@videolan.org" target="_blank">jpsaman@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Angelo,<br><div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote"><div class="im">
On Wed, Jan 23, 2013 at 3:16 PM, Angelo Schiavone <span dir="ltr"><<a href="mailto:angelo.schiavone@gmail.com" target="_blank">angelo.schiavone@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">here is the git patch.</blockquote></div><div><br>In the end I did not use your patches to fix the issues. I just pushed changes that should fix all<br>

the bugs you noticed. It does need some wider testing so please try it ;)<br><br></div><div>If you find any issue, then please report them again.<br></div><div><br></div><div>Some feedback on your patch:<br></div><div>- both NIT and BAT table already start at the right location since p_byte is defined as:<br>
</div></div></div></div></div></blockquote><div><br></div><div>p_byte = p_section->p_playload_start + 2;<br> <br></div><div>  - the other change in BAT table was correctly spotted<br></div><div>  - you were right about the CAT table<br>
</div><div>  - the CRC32 validation for TOT table has been solved in another way.<br><br></div></div>Kind regards,<br>Jean-Paul Saman<br></div></div></div></div>