man: Use -compact also for alternative functions in libbsd(7)

The other function lists use the compact mode, let's be uniform about
this.

getentropy: Fix function cast for getauxval()

Fix also the Hurd implementation, missed in the original fix.

Extends: commit 72a82ee262795c9162e712885381a111fcf988bc
Warned-by: gcc

Release libbsd 0.11.6

Add LoongArch support to nlist()

Closes: !13
Signed-off-by: WANG Xuerui <git@xen0n.name>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

man: Document when functions were added to musl in libbsd(7)

The explicit_bzero(3) and reallocarray(3) functions are also present on
musl libc, mention it in the superseded section.

Remove arc4random() OpenBSD support

The arc4random() support for OpenBSD does not build. The intention was
to include any portability code so that the library could be easily
ported to such systems, but it makes little sense to build it on OpenBSD
where most of the functionality will be already present, or the software
would have been ported anyway.

[guillem@hadrons.org: Reword commit message to add rationale. ]

Closes: !15
Signed-off-by: Guillem Jover <guillem@hadrons.org>

build: Add alpine to CI for tests with musl libc

Closes: !18
Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Import explicit_bzero and strtonum test cases from OpenBSD

- Remove trailing spaces.
- Declare file-scope functions and variables static.
- Declare functions with a proper prototype.
- Do not mix declarations and code for C90 conformance.
- Do not compare size_t and ssize_t variables.

build: Sort test programs in .gitignore

test: Add fpurge() test case using open_memstream(3) FILE

Signed-off-by: Guillem Jover <guillem@hadrons.org>

fpurge: Make it work with non fd based FILEs

Streams opened with for example open_memstream(3) will have no associated
file descriptor, and fileno(3) will fail.

Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Remove static FILE declaration from fpurge test

This does not work with libcs that do not declare the structure
in a header file, like musl. And gets in the way of supporting
non-fd based streams.

Signed-off-by: Guillem Jover <guillem@hadrons.org>

Release libbsd 0.11.5

Revert "build: Add missing proctitle unit test program"

This reverts commit beafad2657c7a57109c28f8bad9cb028c84c7dd5.

This test was already handled later on as part of the BUILD_LIBBSD_CTOR
conditional. Adding it to the unconditional set made the build fail when
the system does not have GNU .init_array section support.

Closes: #9
Analysis-by: Duncan Overbruck <mail@duncano.de>

Document how to build from git and from a tarball release

Closes: #8

build: Check for objdump explicitly

We should not assume that something will implicitly check for this tool,
as we need it ourselves, and this is an internal implementation detail
of right now libtool.

Fixes: commit f11ab6722367f1cf62704ed3c827b9b68dcb5397

build: Respect $(OBJDUMP)

We already search for it in `./configure` so let's respect
the result of that search.

This helps with cross-compilation and any other cases
where one might want to choose a different toolchain.

Closes: !16
Bug: https://bugs.gentoo.org/831863
Signed-off-by: Sam James <sam@gentoo.org>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Release libbsd 0.11.4

man: Add missing comma in .Dd for tree(3)

Reported-by: Thorsten Glaser <tg@mirbsd.de>

man: Update tree(3) man page from OpenBSD

This man page has a BSD-3-clause-author license instead of the old
BSD-4-clause license.

build: Clarify link-time warnings

Mention the involved function instead of stating an implicit "this
function". Mention libbsd when proposing using an alternative libmd
to make the context clear.

Switch md5 compatibility logic back to direct linking

When using the recent dlsym() based wrapper, we are not requiring any
symbol from libmd, as we resolve those dynamically at run-time. We were
ending up linking against libmd because in another part of the code we
require (depending on the architecture) the SHA512 functions for the
getentropy() local implementation. But that function might be provided
by the system libc on some systems, which means we end up not linking
against libmd at all.

To solve this we go back to the previous simpler solution of linking
directly, which had the main drawback of then making programs fail to
link when not specifying -lmd (on platforms that need it). And then
switch the .so link point from a symlink to a linker script, so that we
can inject the -lmd library as-needed. This is similar to what glibc is
doing.

Fixes: commit 31f034e3862debda8615a449b1c11c4d6920dcc7

build: Split libmd dependency due to MD5 functions from SHA requirements

To be able to rework the md5 deprecation logic, we need to detangle when
we depend on libmd due to requiring MD5 functions, which might be
otherwise provided by libc, or when we require SHA functions for the
internal getentropy() implementation.

build: Add new libbsd_symver_weak() macro

We will use it for the reworked md5 deprecation logic.

test: Check that strdup() does not fail

Warned-by: gcc

getentropy: Add missing prototype for BSD variant

Warned-by: gcc

nlist: Remove condition which is always true

The nlist() function is limited to handle ELF binaries of the same class
as size as the size_t of the architecture built.

In addition the SIZE_T_MAX macro is BSD specific, and was falling back
to the wrong constant on most 64-bit non-BSD systems.

Warned-by: gcc

build: Append __ after __attribute and __typeof keywords

Be consistent with other usages in the code base.

build: Ignore backup files

test: Explicitly mark symbols as used

Because some of the symbols are not otherwise referenced, GCC would like
to remove them.

Closes: !14
Signed-off-by: Guillem Jover <guillem@hadrons.org>

build: Enable .init_array support when building with LTO

Because these symbols are not otherwise referenced, GCC would like
to remove them.

Signed-off-by: Guillem Jover <guillem@hadrons.org>

man: Fix .Nx macro usage

Its arguments are used as version numbers, so we need continue the
content on the next line.

build: Allow building with -flto on gcc-10 and newer

Global asm statements (like .symver directives) do not work reliably
in gcc with link time optimization. Use the symver attribute introduced
with gcc-10 to set symbol versions instead, if available.

[guillem@hadrons.org:
 - Simplify by using __has_attribute fallback from <sys/cdefs.h>.
 - Coding style changes. ]

Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Fix pipe_feed() to allow checking fprintf format strings

Warned-by: gcc -W

build: Add missing proctitle unit test program

test: Do not pass NULL as the first funopen() argument

Warned-by: gcc -W

build: Detect sed at configure time

Check whether sed is available and use the implementation matching the
requirements via the SED variable.

vis: Add prototypes for strnvis() and strnunvis() variants

Warned-by: gcc

test: Cast literal strings to (char *) on n_name assignment

The member is declared as n_name so we cannot directly assign a literal
string constant.

Warned-by: gcc

Add missing prototypes to functions

Warned-by: gcc

build: Add compiler warnings support

Detect as many warnings as possible during configure and enable them
if the user did not supply any, so that any such problem can be spotted
and fixed.

nlist: Remove repeated shadowing variable declaration

Warned-by: gcc

getentropy: Fix function cast for getauxval()

Warned-by: gcc

test: Add proper prototypes for main() function

Warned-by: gcc

Mark local functions as static

Warned-by: gcc

Add link-time warnings to MD5 wrapper functions

Let's get the word out that these functions are deprecated and should be
switched away from.

Require a semicolon for libbsd_link_warning() macro

Remove the semicolon in the macro definition to force adding one on the
call sites, to make the code look like an actual function.

man: Call the libbsd-ctor library by its name instead of bsd-ctor

man: Fix pkg-config(1) references in libbsd(7) man page

build: Add code coverage support in the GitLab CI

build: Use apt-get instead of apt in the GitLab CI

The former is to be used programmatically, while the latter is intended
for interactive use.

build: Add a Libs.private field to overlay pkg-config file

We need to list all internal libraries there so that we can statically
link.

Fixes: commit 2374f409defb380d0c5c07f28b9c166ef8bdc742

Release libbsd 0.11.3

Switch libmd wrapper to use dlsym()

Switch from the previous versioned symbol implementation which required
users to also link against the message digest provider explicitly, or
they would fail to find the symbols, to an implementation that loads
the symbols from the linked library providing the functions using
dlsym(), thus preserving backwards compatibility.

build: Add a Libs.private field to pkg-config file

We need to list all internal libraries there so that we can statically
link.

build: Use a single variable to track libraries to link against

Using various variables means we have to keep these in sync in various
places. Just use a single variable that we can use anywhere where this
is needed.

build: Fix message digest library checks

They were not failing when not finding the SHA-2 functions and
were hardcoding -lmd regardless of what library had been found.

Use uintptr_t and size_t instead of __-prefixed types in <sys/cdefs.h>

The __-prefixed types cannot be assumed to be defined. Use the standard
types instead.

Closes: #6

Release libbsd 0.11.2

closefrom: Use close_range() on Linux when available

Closes: !11
Based-on-patch-by: cptpcrd <cptpcrd.git@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

closefrom: Handle lowfd < 0 properly

More important if close_range() is going to be used, since casting
negative values to 'unsigned int' might hide the errors.

[guillem@hadrons.org: Minor coding style fix. ]

Signed-off-by: Guillem Jover <guillem@hadrons.org>

closefrom: Import some changes from sudo

Take most of the changes done in sudo, but preserve the existing local
changes and refactoring.

In addition, refactor pstat implementation into closefrom_pstat(), so
that the code is easier to read, and requires no conditional
declarations.

Update <sys/queue.h> from FreeBSD

This brings <sys/queue.h> to the most up-to-date version from FreeBSD,
incorporating 18 commits from the past 5 years (2015-02-24 - 2021-01-25):

  $ git log --oneline 9090a24aed70..8d55837dc133 sys/sys/queue.h share/man/man3/queue.3

Only minimal changes compared to the FreeBSD version have been applied
(queue.3 -> queue.3bsd, _LIBBSD_ prefix).

[guillem@hadrons.org: Remove reference to kernel mode in man page. ]

Closes: !12
Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Improve code coverage for strnstr(3) unit tests

man: Update libbsd(7) man page with updates in 0.11.0

build: Install libmd-dev in the gitlab CI

build: Switch gitlab CI to use a Debian buster

Release libbsd 0.11.1

Add support for new LIBBSD_VIS_OPENBSD selection macro

This will make it possible to explicitly select the OpenBSD vis
implementation (the current default) for code of OpenBSD origins.

Release libbsd 0.11.0

Update copyright claims

Use libmd hashing function implementations instead of embedding our own

This splits the implementation responsibilities, and reduces embedded
code copies, which was one of the driving points with this project to
start with, so it's nice to give a good example.

Provide a default progname on Windows

[guillem@hadrons.org:
 - Remove .exe extension from default program name.
 - Call reallocarray() once by switching to a «do {} while» loop.
 - Minor coding style fixes. ]

Signed-off-by: Guillem Jover <guillem@hadrons.org>

Import pwcache module from OpenBSD

man: Remove empty line in reallocarray(3bsd)

Warned-by: lintian
Fixes: commit 01f0d1ea1e71f1018a009ebd9203dd48e6d90c45

Add recallocarray() and freezero() from OpenBSD

Add recallocarray(), introduced in OpenBSD 6.1, and freezero(),
introduced in OpenBSD 6.2. The former is imported as-is from OpenBSD,
while the latter is the non-malloc-internal branch of the same code (and
also the OpenSSH portable variant).

Both of these originated in OpenBSD, but have also been implemented by
IllumOS, cf. https://www.illumos.org/issues/8546

Documentation for these functions is in malloc(3) upstream, the relevant
parts of which were previously imported in reallocarray(3bsd). Update
reallocarray(3bsd) with the changes that were introduced since, and add
the relevant bits for recallocarray() and freezero(), plus aliases.

[guillem@hadrons.org: Update copyright in COPYING. ]

Closes: !10
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Fix ELF detection on Intel compilers

The Intel compiler does not define __amd64__ on x86_64 platforms;
instead, like other compilers, it defines __x86_64__ .

Closes: !8
Signed-off-by: Guillem Jover <guillem@hadrons.org>

man: Add a timespec(3bsd) alias to timeval(3bsd)

Even though man-pages project now includes man pages for system data
types, we still include these for any other system that does not have
them, to provide a self-contained project with code and documentation.

man: Add missing LIBRARY section

Fix coding style

test: Fix short lived memory leaks

These are non-issues, but having a clean ASAN test suite makes it
possible to detect actual problems in the tested code.

Warned-by: gcc ASAN

funopen: Fix memory leak in funopen_close() when closefn is NULL

We need to free the cookiewrap even when the closefn method is NULL.

Warned-by: gcc ASAN

build: Detect support for --version-script in ld

Do not define SIZE_T_MAX if already defined

man: Add missing doc-operating-system-NetBSD string variable

Fixes: commit 99320b9168ffb0112def1a712e8d59441d1b46d9.

Add missing strnvisx() to the symbols script

Fixes: commit 2d7de186e9cb19a756c0630ee85cb3f2d29b3484.

Release libbsd 0.10.0

err: Add err(), warn(), errx() and warnx() familiy of functions

Some systems such as Windows or musl-libc based ones do not have these
BSD extensions. In addition libbsd itself is making use of the warnx()
functions, so we better provide these interfaces in case they are
missing.

err: Rewrite warnc() and errc() family functions to be standalone

Do not depend on the system vwarn() and verr() functions to implement
the *c() variants, as the system might actually lack any of the <err.h>
BSD extensions.

err: Mark error functions as non-returning with __dead2

Add e2k support for nlist()

This is a Russian 64-bit LE VLIW architecture named Elbrus
(formerly Elbrus 2000).

[guillem@hadrons.org:
 - Place the entry in alphabetical order. ]

Signed-off-by: Guillem Jover <guillem@hadrons.org>

Add AArch64 ILP32 support to nlist()

Closes: !7
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Add ARC support to nlist()

Closes: !6
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Add a comment to note the ELF entries are sorted alphabetically

This should help people wanting to add new entries.

Re-allow direct use of nlist.n_name in <nlist.h>

Commit e8d340de ("Remove a.out support from nlist()") introduced a copy
of the definition of nlist from a.out.h. However, as well as having
n_name inside n_un, on the various BSDs n_name could also be accessed
as a direct member of nlist, and this is made use of by FreeBSD's
usr.bin/netstat/main.c. Thus we should also add the same enclosing
anonymous union.

[guillem@hadrons.org:
 - Add a minimal unit test. ]

Closes: !4
Signed-off-by: Guillem Jover <guillem@hadrons.org>

nlist: Fix out-of-bounds read on strtab

When doing a string comparison for a symbol name from the string table,
we should make sure we do a bounded comparison, otherwise a non-NUL
terminated string might make the code read out-of-bounds.

Warned-by: coverity

nlist: Fix unbounded malloc() calls

There are a couple of malloc() calls with unbounded size arguments,
coming from the parsed file. We need to make sure the size is not
larger than the file being parsed, otherwise we might end up with
out of memory conditions.

Reported-by: Daniel Hodson <daniel@elttam.com.au>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

nlist: Fix pread() return value check

We should check for partial reads, and not continue in those cases,
as we are not retrying them, otherwise we might end up operating on
uninitialized data.

Reported-by: Daniel Hodson <daniel@elttam.com.au>
Based-on-patch-by: Daniel Hodson <daniel@elttam.com.au>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

nlist: Check whether sh_link is within bounds

The sh_link members should be >= e_shnum, otherwise we might do out of
bounds read accesses on the shdr array.

Reported-by: Daniel Hodson <daniel@elttam.com.au>
Based-on-patch-by: Daniel Hodson <daniel@elttam.com.au>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

nlist: Check that e_shnum and e_shentsize are within bounds

The e_shnum must not be 0, otherwise we will do a zero sized allocation
and further processing of the executable will lead to out of bounds
read/write accesses. The e_shentsize must be equal to sizeof(Elf_Shdr),
otherwise we will perform out of bounds read accesses on the shdr array.

Reported-by: Daniel Hodson <daniel@elttam.com.au>
Based-on-patch-by: Daniel Hodson <daniel@elttam.com.au>
Signed-off-by: Guillem Jover <guillem@hadrons.org>