Bug 200218

Summary:	app-crypt/trousers-0.3.1 build failure on 64bit arch due to pointer to integer cast
Product:	Gentoo Linux	Reporter:	Tiziano Müller (RETIRED) <dev-zero>
Component:	New packages	Assignee:	Crypto team [DISABLED] <crypto+disabled>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	trousers-0.3.1-qa.patch

Description Tiziano Müller (RETIRED) gentoo-dev

2007-11-24 18:13:05 UTC

x86_64-pc-linux-gnu-gcc -DPACKAGE_NAME=\"trousers\" -DPACKAGE_TARNAME=\"trousers\" -DPACKAGE_VERSION=\"0.3.1\" "-DPACKAGE_STRING=\"trousers 0.3.1\"" -DPACKAGE_BUGREPORT=\"trousers-tech@lists.sf.net\" -DPACKAGE=\"trousers\" -DVERSION=\"0.3.1\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_OPENSSL_BN_H=1 -DHAVE_OPENSSL_ENGINE_H=1 -DHAVE_PTHREAD_H=1 -DHAVE_DLFCN_H=1 -I. -I. -I../include -DAPPID=\"TSPI\" -DVAR_PREFIX=\"/var\" -DETC_PREFIX=\"/etc\" -DTSS_BUILD_TSS12 -DTSS_BUILD_TRANSPORT -DTSS_BUILD_TICK -DTSS_BUILD_COUNTER -DTSS_BUILD_AUTH -DTSS_BUILD_GETSET -DTSS_BUILD_RANDOM -DTSS_BUILD_CAPS -DTSS_BUILD_CAPS_TPM -DTSS_BUILD_POLICY -DTSS_BUILD_DIR -DTSS_BUILD_PCR_EVENTS -DTSS_BUILD_HASH -DTSS_BUILD_SIGN -DTSS_BUILD_QUOTE -DTSS_BUILD_PCR_COMP -DTSS_BUILD_SEAL -DTSS_BUILD_CHANGEAUTH -DTSS_BUILD_BIND -DTSS_BUILD_OWN -DTSS_BUILD_PS -DTSS_BUILD_ADMIN -DTSS_BUILD_AIK -DTSS_BUILD_EK -DTSS_BUILD_CERTIFY -DTSS_BUILD_KEY -DTSS_BUILD_MAINT -DTSS_BUILD_MIGRATION -DTSS_BUILD_PCR_EXTEND -DTSS_BUILD_SELFTEST -DTSS_BUILD_PCRS_LIST -DTSS_BUILD_HASH_LIST -DTSS_BUILD_ENCDATA_LIST -DTSS_BUILD_RSAKEY_LIST -DTSS_BUILD_ASN1 -DTSS_BUILD_AUDIT -DTSS_BUILD_SEALX -DTSS_BUILD_QUOTE2 -I/usr/include/gtk-2.0 -I/usr/lib64/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/libpng12 -DTSS_BUILD_NV -DTSS_BUILD_DELEGATION -DTSS_BUILD_CMK -march=nocona -O2 -pipe -m64 -DBI_OPENSSL -I../include -W -Wall -Werror -Wno-unused-parameter -DTCSD_DEFAULT_PORT=30003 -DTSS_VER_MAJOR=0 -DTSS_VER_MINOR=3 -DTSS_SPEC_MAJOR=1 -DTSS_SPEC_MINOR=2 -MT libtspi_la-tspi_nv.lo -MD -MP -MF .deps/libtspi_la-tspi_nv.Tpo -c tspi_nv.c  -fPIC -DPIC -o .libs/libtspi_la-tspi_nv.o
cc1: warnings being treated as errors
tspi_nv.c: In function 'Tspi_NV_ReleaseSpace':
tspi_nv.c:220: warning: cast from pointer to integer of different size
make[2]: *** [libtspi_la-tspi_nv.lo] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory `/var/tmp/paludis/app-crypt/trousers-0.3.1/work/trousers-0.3.1/src/tspi'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/paludis/app-crypt/trousers-0.3.1/work/trousers-0.3.1/src'
make: *** [all-recursive] Error 1

Comment 1 Tiziano Müller (RETIRED) gentoo-dev

2007-11-24 18:14:34 UTC

Usually, I'd say that -Werror should be stripped, but with such software, I'd say that it should be reported to upstream and fixed properly...

Comment 2 Alon Bar-Lev (RETIRED) gentoo-dev

2007-11-24 19:14:00 UTC

Created attachment 136901 [details, diff]
trousers-0.3.1-qa.patch

Can you please try the following patch?
Looking at the source, TSS_HPCRS is TSS_HOBJECT which in turn is UINT32.
I cannot compile 64bit, so if you have more of these we can send patch to upstream.

Comment 3 Tiziano Müller (RETIRED) gentoo-dev

2007-11-24 22:23:58 UTC

Works perfectly. Thanks.

Comment 4 Alon Bar-Lev (RETIRED) gentoo-dev

2007-11-24 22:36:39 UTC

Thanks!