<p>On 16/08/31 22:33, 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> Picture dimensions should never be zero, since most filters and outputs
 will not deal with it.</code></pre>
</blockquote>
<ul>
<li>Do we have a relevant <code>assert</code> somewhere that is triggered when the dimensions are zero?</li>
</ul>
<p>I think that would be a viable addition to the patch, or perhaps state somewhere in the documentation that dimensions that includes such value results in <em>undefined-behavior</em> (together with an <code>assert</code>).</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> If they were, this code would leak the picture pool in under a second
 anyway.

 This reverts commit e62911abc7df0de022a467f3e5b373f324cc4078.
 ---
  modules/video_filter/scale.c | 8 --------
  1 file changed, 8 deletions(-)

 diff --git a/modules/video_filter/scale.c b/modules/video_filter/scale.c
 index b06b4f0..818aa28 100644
 --- a/modules/video_filter/scale.c
 +++ b/modules/video_filter/scale.c
 @@ -90,14 +90,6 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic )

      if( !p_pic ) return NULL;

 -    if( (p_filter->fmt_in.video.i_height == 0) ||
 -        (p_filter->fmt_in.video.i_width == 0) )
 -        return NULL;
 -
 -    if( (p_filter->fmt_out.video.i_height == 0) ||
 -        (p_filter->fmt_out.video.i_width == 0) )
 -        return NULL;
 -
      video_format_ScaleCropAr( &p_filter->fmt_out.video, &p_filter->fmt_in.video );

      /* Request output picture */
 -- 
 2.9.3

 _______________________________________________
 vlc-devel mailing list
 To unsubscribe or modify your subscription options:
 https://mailman.videolan.org/listinfo/vlc-devel</code></pre>
</blockquote>