<!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>Dear Filip,</p>
<ul>
<li>Review your patches <em>before</em> sending to <code>vlc-devel</code>, not after.</li>
<li>…</li>
<li>Yes, I know you cannot smoke in front of your computer.</li>
<li>…</li>
<li>Aw come on, you can definitely have a smoke before sending, instead of after.</li>
<li>…</li>
<li>No, I do not care that the coffee is too hot to drink and that you’d like a cup while smoking.</li>
</ul>
<p>On 2017-05-25 21:58, Filip Roséen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> This allows for easier implementation for modules using the
 concat-list, as it can assume that all entries can be passed directly
 to functions such as vlc_stream_NewURL (as they are now valid URIs).

 These changes also makes it so that the contents of concat-list
 matches that of the documentation, as it uses the notion "input URLs".
 ---
  src/input/input.c | 27 +++++++++++++++------------
  1 file changed, 15 insertions(+), 12 deletions(-)

 diff --git a/src/input/input.c b/src/input/input.c
 index f68bc7e631..479dd50238 100644
 --- a/src/input/input.c
 +++ b/src/input/input.c
 @@ -2845,7 +2845,6 @@ static void InputGetExtraFilesPattern( input_thread_t *p_input,
  {
      int i_list;
      char **ppsz_list;
 -
      TAB_INIT( i_list, ppsz_list );

      char *psz_base = strdup( psz_path );
 @@ -2860,24 +2859,28 @@ static void InputGetExtraFilesPattern( input_thread_t *p_input,
      /* Try to list files */
      for( int i = i_start; i <= i_stop; i++ )
      {
 -        struct stat st;
 -        char *psz_file;
 -
 -        if( asprintf( &psz_file, psz_format, psz_base, i ) < 0 )
 +        char *psz_probe;
 +        if( asprintf( &psz_probe, psz_format, psz_base, i ) < 0 )
              break;

 -        char *psz_tmp_path = get_path( psz_file );
 +        char *psz_path = get_path( psz_probe );</code></pre>
</blockquote>
<p>Right when I sent this email I noticed that I did not fix a previous, potential, null-dereference (on the line above) as part of the refactoring.. bloody shameful.</p>
<p>Attached is the equivalent patch where this is taken care of.</p>
<p>Best Regards,<br />
Filip</p>
</body>
</html>