Hi,

On Tue, Jul 26, 2016 at 04:09:16AM +0200, Jan Just Keijser wrote:
> > Anyone having a good argument against it?  JJK, do you happen to know
> > what this is about?
> 
> this is from even before my time ;)

Fun :-) 

> I didn't even know that you could specify cipher without an argument - 
> let's disable this (and I assume the same is true for --auth ?)

Indeed, the same is true, and it would lead to the same crash
(--auth none --auth)

  else if (streq (p[0], "auth") && p[1] && !p[2])
    {
      VERIFY_PERMISSION (OPT_P_GENERAL);
      options->authname_defined = true;
      options->authname = p[1];
      if (streq (options->authname, "none"))
        {
          options->authname_defined = false;
          options->authname = NULL;
        }
    }
  else if (streq (p[0], "auth") && !p[1])
    {
      VERIFY_PERMISSION (OPT_P_GENERAL);
      options->authname_defined = true;
    }

I think Steffan's patch (that he sent in parallel to my mail) takes care
of this as well...

gert

-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Attachment: signature.asc
Description: PGP signature

Reply via email to