Skip to content
Snippets Groups Projects
  1. Dec 05, 2014
  2. Jul 16, 2014
    • David Woodhouse's avatar
      Remove all traces of FBOpenSSL SPNEGO support · 9ad282b1
      David Woodhouse authored
      This is just fundamentally broken. SPNEGO (RFC4178) is a protocol which
      allows client and server to negotiate the underlying mechanism which will
      actually be used to authenticate. This is *often* Kerberos, and can also
      be NTLM and other things. And to complicate matters, there are various
      different OIDs which can be used to specify the Kerberos mechanism too.
      
      A SPNEGO exchange will identify *which* GSSAPI mechanism is being used,
      and will exchange GSSAPI tokens which are appropriate for that mechanism.
      
      But this SPNEGO implementation just strips the incoming SPNEGO packet
      and extracts the token, if any. And completely discards the information
      about *which* mechanism is being used. Then we *assume* it was Kerberos,
      and feed the token into gss_init_sec_context() with the default
      mechanism (GSS_S_NO_OID for the mech_type argument).
      
      Furthermore... broken as this code is, it was never even *used* for input
      tokens anyway, because higher layers of curl would just bail out if the
      server actually said anything *back* to us in the negotiation. We assume
      that we send a single token to the server, and it accepts it. If the server
      wants to continue the exchange (as is required for NTLM and for SPNEGO
      to do anything useful), then curl was broken anyway.
      
      So the only bit which actually did anything was the bit in
      Curl_output_negotiate(), which always generates an *initial* SPNEGO
      token saying "Hey, I support only the Kerberos mechanism and this is its
      token".
      
      You could have done that by manually just prefixing the Kerberos token
      with the appropriate bytes, if you weren't going to do any proper SPNEGO
      handling. There's no need for the FBOpenSSL library at all.
      
      The sane way to do SPNEGO is just to *ask* the GSSAPI library to do
      SPNEGO. That's what the 'mech_type' argument to gss_init_sec_context()
      is for. And then it should all Just Work™.
      
      That 'sane way' will be added in a subsequent patch, as will bug fixes
      for our failure to handle any exchange other than a single outbound
      token to the server which results in immediate success.
      9ad282b1
  3. May 11, 2013
  4. Feb 09, 2013
  5. Jan 28, 2013
  6. Jul 03, 2012
  7. May 27, 2012
  8. May 22, 2012
  9. Apr 26, 2012
  10. Apr 20, 2012
  11. Dec 30, 2011
  12. Sep 25, 2011
  13. Sep 23, 2011
  14. Sep 21, 2011
  15. Sep 20, 2011
  16. Mar 11, 2011
  17. Mar 24, 2010
  18. Sep 03, 2009
  19. May 09, 2008
  20. Apr 02, 2008
  21. Mar 31, 2008
  22. Feb 18, 2008
  23. Jan 29, 2008
  24. Jan 17, 2008
  25. Aug 25, 2007
  26. Aug 24, 2007
  27. Aug 21, 2007
  28. Aug 20, 2007
  29. Aug 19, 2007
  30. Aug 14, 2007
  31. Aug 08, 2007
  32. Aug 03, 2007
Loading