#1469 7.40.0: GSSAPI does not compile on NetBSD

[Thomas Klausner]

On NetBSD-7.99.3/amd64, curl-7.40.0 does not support the built-in GSSAPI any longer. 7.39.0 worked fine.

The error message is

/bin/sh ../libtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I../include/curl -I../include -I../include -I../lib -I../lib -DBUILDING_LIBCURL -DCURL_HIDDEN_SYMBOLS -I/usr/include/krb5 -I/usr/include -I/usr/pkg/include -I/usr/include -I/usr/include/gssapi -I/usr/include/openssl -fvisibility=hidden -O2 -I/usr/include/krb5 -I/usr/include -I/usr/pkg/include -Wno-system-headers -MT libcurl_la-curl_sasl_gssapi.lo -MD -MP -MF .deps/libcurl_la-curl_sasl_gssapi.Tpo -c -o libcurl_la-curl_sasl_gssapi.lo test -f 'curl_sasl_gssapi.c' || echo './'curl_sasl_gssapi.c
libtool: compile: gcc -DHAVE_CONFIG_H -I../include/curl -I../include -I../lib -DBUILDING_LIBCURL -DCURL_HIDDEN_SYMBOLS -I/usr/include/krb5 -I/scratch/www/curl/work/.buildlink/include -I/usr/include/gssapi -I/usr/include/openssl -fvisibility=hidden -O2 -Wno-system-headers -MT libcurl_la-curl_sasl_gssapi.lo -MD -MP -MF .deps/libcurl_la-curl_sasl_gssapi.Tpo -c curl_sasl_gssapi.c -fPIC -DPIC -o .libs/libcurl_la-curl_sasl_gssapi.o
curl_sasl_gssapi.c: In function 'Curl_sasl_create_gssapi_user_message':
curl_sasl_gssapi.c:129:40: error: 'gss_nt_service_name' undeclared (first use in this function)
gss_nt_service_name, &krb5->spn);
^
curl_sasl_gssapi.c:129:40: note: each undeclared identifier is reported only once for each function it appears in
*** [libcurl_la-curl_sasl_gssapi.lo] Error code 1

[Steve Holme]

Thank you for your bug report.

curl_sasl_gssapi.c is a new file in 7.40.0 so you wouldn't see the compilation error in previous versions.

As I don't have access to a NetBSD installation, are you able to modify line 129 from using gss_nt_service_name to GSS_C_NT_HOSTBASED_SERVICE to see if that fixes the issue?

It seems I may have copied that line from socks_gssapi.c, which does some extra pre-processor magic, rather than http_negoitate.c.

Kind Regards

Steve

[Thomas Klausner]

Indeed, the suggested change fixes the build. Thanks for the quick reply.

For the record I've attached the patch I'm using.

[Steve Holme]

Many thanks.

I have updated the patch a little and pushed as commit 5c0e66d632.