Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Sep 2021 23:48:47 +0300
From:      Toomas Soome via freebsd-current <freebsd-current@freebsd.org>
To:        FreeBSD User <freebsd@walstatt-de.de>
Cc:        FreeBSD CURRENT <freebsd-current@freebsd.org>
Subject:   Re: FreeBSD-13-STABLE: lib/libsecureboot/verify_file.c: error: use of undeclared identifier 'SOPEN_MAX'
Message-ID:  <18682E1D-E64A-4F02-8949-491763DCD9A2@me.com>
In-Reply-To: <20210903180022.41ad35ae@thor.intern.walstatt.dynvpn.de>
References:  <20210903180022.41ad35ae@thor.intern.walstatt.dynvpn.de>

next in thread | previous in thread | raw e-mail | index | archive | help



> On 3. Sep 2021, at 18:59, FreeBSD User <freebsd@walstatt-de.de> wrote:
> 
> Hello,
> 
> enabling 
> 
> WITH_BEARSSL 
> 
> in src.conf renders buildworld on 13-STABLE to fail, but not on
> 14-CURRENT. 
> 
> 
> 
> This is the difference between the sources, obviously 14-CURRENT contains the correct
> definition of SOPEN_MAX, while 13-STABLE not (undefinied SOPNE_MAX triggers the compiler to
> fail, 
> /usr/src/lib/libsecureboot/verify_file.c:59:22: error: use of undeclared identifier
> 'SOPEN_MAX'), see
> 
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=258211
> 
> 
> [...]
> 13-STABLE
> :/pool/sources/13-STABLE/src # grep -r SOPEN_MAX .
> ./lib/libsecureboot/tests/Makefile:XCFLAGS.verify_file += -DSOPEN_MAX=64
> ./lib/libsecureboot/verify_file.c:static int ve_status[SOPEN_MAX+1];
> ./lib/libsecureboot/verify_file.c:      if (fd >= 0 && fd < SOPEN_MAX) {
> ./lib/libsecureboot/verify_file.c:      ve_status[SOPEN_MAX] = ves;
> ./lib/libsecureboot/verify_file.c: *    @li ve_status[SOPEN_MAX] if ve_status_state is none
> ./lib/libsecureboot/verify_file.c:              fd >= 0 && fd < SOPEN_MAX)
> ./lib/libsecureboot/verify_file.c:      return (ve_status[SOPEN_MAX]);  /* most recent */
> 
> [...]
> 14-CURRENT
> ./lib/libsecureboot/tests/Makefile:XCFLAGS.verify_file += -DSOPEN_MAX=64
> ./lib/libsecureboot/verify_file.c:#ifndef SOPEN_MAX
> ./lib/libsecureboot/verify_file.c:#define       SOPEN_MAX       64
> ./lib/libsecureboot/verify_file.c:static int ve_status[SOPEN_MAX+1];
> ./lib/libsecureboot/verify_file.c:      if (fd >= 0 && fd < SOPEN_MAX) {
> ./lib/libsecureboot/verify_file.c:      ve_status[SOPEN_MAX] = ves;
> ./lib/libsecureboot/verify_file.c: *    @li ve_status[SOPEN_MAX] if ve_status_state is none
> ./lib/libsecureboot/verify_file.c:              fd >= 0 && fd < SOPEN_MAX)
> ./lib/libsecureboot/verify_file.c:      return (ve_status[SOPEN_MAX]);  /* most recent */
> 
> 
> 
> 
> -- 
> O. Hartmann
> 

Hi!

Sorry, it is fixed now. Missed one cherry-pick.

rgds,
toomas


Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?18682E1D-E64A-4F02-8949-491763DCD9A2>