[Swan-commit] Changes to ref refs/heads/master

[D. Hugh Redelmeier]

New commits:
commit 68051c2ac91a52a87cad2e6adbaa0a96580d9c1e
Author: D. Hugh Redelmeier <hugh at mimosa.com>
Date:   Mon Nov 5 10:45:15 2018 -0500

    pluto: tidy fetch.c
    
    - simplify conditional compilation (#if) structure
    
    - preserve "why" with crl_fetch_list_mutex and check
      that lock and unlock match
    
    - handle realloc failure
    
    - exploit DBGF, clone_chunk, clone_bytes_as_chunk, chunk_eq
    
    - DBG_log more information about libcurl failure.  Should this go into
      the real log?
    
    - reduce scope of autos; combine declaration with initialization
    
    - convert some while-loops to for-loops, clarifying the control
      variable use
    
    - absorb code into for-body when that avoids testing a condition twice
      or eliminates a bool.
    
    - add a clarifying const
    
    - simplify control flow in fetch_ldap_url.  For example, eliminate
      rewriting and retesting rc.  It sure would be handy if the manual
      for ldap_result(3) explained the return value.
    
    - in insert_crl_nss: avoid allocating and freeing a string for a
      missing URI.