FTBFS on older arm

0.3.5.1-alpha failed to build on arm64/armhf/armel jessie, stretch (only arm64), trusty, xenial, and artful.

Looking at some of the logs, this comes up repeatedly:

src/lib/libtor-crypt-ops.a(src_lib_libtor_crypt_ops_a-aes_openssl.o): In function `aes_set_iv':
./build/../src/lib/crypt_ops/aes_openssl.c:399: undefined reference to `get_uint32'
./build/../src/lib/crypt_ops/aes_openssl.c:400: undefined reference to `get_uint32'
./build/../src/lib/crypt_ops/aes_openssl.c:401: undefined reference to `get_uint32'

https://jenkins.torproject.org/view/packages%20-%20debian/job/tor-debian-release-binaries-arm/44/

Cheers,

changed milestone to %Tor: 0.3.5.x-final

added 034-backport component::core tor/tor milestone::Tor: 0.3.5.x-final priority::medium resolution::fixed reviewer::dgoulet severity::blocker status::closed tor-arm type::defect version::tor 0.3.5.1-alpha labels

What compiler version is this?

In 0.3.5, we turned those functions into inline functions in the bytes.h header, since they're used all over and they're great inlining candidates.

I wonder if this is one of those things where old compilers have problems with inline functions and linking them when it decides not to inline them.

Trac:
Milestone: N/A to Tor: 0.3.5.x-final

I don't know for certain, but I'd guess the stretch build uses 6.3, the jessie build 4.9.

Oh, or maybe this is a big-endian issue. The AES-CTR code is slightly different on big-endian systems.

According to https://wiki.debian.org/ArchitectureSpecificsMemo, all the arms are LE

also maybe related:

15:00:32 ../src/lib/crypt_ops/aes_openssl.c: In function 'aes_cipher_free_':
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:338:3: warning: implicit declaration of function 'memwipe'; did you mean 'mempcpy'? [-Wimplicit-function-declaration]
15:00:32    memwipe(cipher, 0, sizeof(aes_cnt_cipher_t));
15:00:32    ^~~~~~~
15:00:32    mempcpy
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:338:3: warning: nested extern declaration of 'memwipe' [-Wnested-externs]
15:00:32 ../src/lib/crypt_ops/aes_openssl.c: In function 'aes_set_iv':
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:399:22: warning: implicit declaration of function 'ntohl'; did you mean 'atoll'? [-Wimplicit-function-declaration]
15:00:32    cipher->counter3 = ntohl(get_uint32(iv));
15:00:32                       ^~~~~
15:00:32                       atoll
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:399:22: warning: nested extern declaration of 'ntohl' [-Wnested-externs]
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:399:28: warning: implicit declaration of function 'get_uint32'; did you mean 'getline'? [-Wimplicit-function-declaration]
15:00:32    cipher->counter3 = ntohl(get_uint32(iv));
15:00:32                             ^~~~~~~~~~
15:00:32                             getline
15:00:32 ../src/lib/crypt_ops/aes_openssl.c:399:28: warning: nested extern declaration of 'get_uint32' [-Wnested-externs]

from https://jenkins.torproject.org/view/packages%20-%20debian/job/tor-debian-release-binaries-arm/44/ARCHITECTURE=arm64,SUITE=artful/consoleFull

Ah. This is happening only on Arm with openssl < 1.1, because on that platform we don't #define USE_EVP_AES_CTR

See branch bug27781, bugfix on https://github.com/torproject/tor/pull/357

Trac:
Status: new to needs_review

This may need a partial backport to 0.3.4; see #27812 (moved)

Trac:
Keywords: N/A deleted, 034-backport added

group [[doc/arm]] issues (using tor-arm to avoid mixing with apparmor keyword, maybe there's a better alternative)

Trac:
Keywords: N/A deleted, tor-arm added

I can't really validate compilation on ARM but the fixes lgtm.

Trac:
Status: needs_review to merge_ready
Reviewer: N/A to dgoulet

Partially backported with f57d8722e04336031e06633cfc493b04d0c16f90; main fix merged to master.

Trac:
Status: merge_ready to closed
Resolution: N/A to fixed

closed

mentioned in issue #27812 (moved)

mentioned in issue #27818 (moved)

moved to tpo/core/tor#27781 (closed)