Page MenuHomeFreeBSD

ftp/curl: Add NTLM option (Fix OpenSSL build w/o DES)
ClosedPublic

Authored by koobs on Nov 22 2019, 10:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 7:00 AM
Unknown Object (File)
Nov 10 2023, 9:17 AM
Unknown Object (File)
Nov 7 2023, 12:26 AM
Unknown Object (File)
Nov 5 2023, 10:29 PM
Unknown Object (File)
Oct 16 2023, 6:27 AM
Unknown Object (File)
Oct 9 2023, 8:14 AM
Unknown Object (File)
Oct 5 2023, 11:23 PM
Unknown Object (File)
Oct 4 2023, 9:28 PM
Subscribers

Details

Summary
ftp/curl: Add NTLM option (Fix OpenSSL build w/o DES)

curl fails to build when OpenSSL is built without DES support, with many
"use of undeclared identifier" errors. For example:

curl_ntlm_core.c:392:3: error: use of undeclared identifier 'DES_key_schedule'
  DES_key_schedule ks;
  ^

An issue was reported back in 2015 [1], requesting support for ./configure
detecting OPENSSL_NO_DES, but the issue was ultimately closed:

"We just don't support very custom OpenSSL builds. I could be talked into
accepting patches that introduce support for this however".

Accordingly and leui of an upstream configure patch, this change adds an
NTLM option, enabled by default, and when disabled, defines
CURL_DISABLE_NTLM, which prevents #define'ing USE_NTLM in
WRKSRC/lib/curl_setup.h, resulting in a successful build.

This allows users the opportunity to easily disable the DES requiring
functionality in curl, if they happen to run custom or stripped down
OpenSSL builds.

Note: This issue may impact builds with other SSL libraries that allow
disabling DES, but I did not investigate this question.

[1] https://sourceforge.net/p/curl/bugs/1439/

Reviewed_by: sunpoet (maintainer)
Approved_by: sunpoet (maintainer)
Test Plan
  • portlint: OK (no delta)
  • testport: OK (poudriere: 12amd64, with/without NTLM option)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable