<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="Content-Style-Type" content="text/css" />
<meta name="generator" content="pandoc" />
<title></title>
<style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Remi,</p>
<p>On 2016-10-29 12:51, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> More extensive fix? It looks like a simple integer overflow.
In general, I am all for clean-ups, but in this case, I think that a simple
fix that can be backported to VLC 2.2 is desirable.</code></pre>
</blockquote>
<p>I wanted to address other issues in the implementation (such as returning <code>VLC_EGENERIC</code> from <code>Open</code> after reading from the <em>source-stream</em>), as well as other <em>integer-overflows</em> (though not as severe as this one, it will “only” lead to distortion during playback).</p>
<p>I am currently at the airport, but I can modify the original patch and submit it before I board (just need to get something to eat first).</p>
<p>Best Regards,<br />
Filip</p>
</body>
</html>