man: Fix ungrammatical construct

Warned-by: lintian

build: Support clock_gettime() provided in librt

In older glibc versions (< 2.17) clock_gettime() is in librt. Add a
check for this to avoid build breakage for programs/libraries that
use libbsd on such systems.

Based-on-patch-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Switch URLs from http or git to https

Fix file descriptor leak in HASHFileChunk helper

This leak only happens on error conditions, so it's not too bad.

Warned-by: coverity

Fix unportable sizeof() usage

We are calculating the size of the array, and need to pass the size of
each element, not the size of a pointer to an element. Although this
happens to be the same in many cases, this is not a portable assumption.

Warned-by: coverity

Release libbsd 0.8.2

Fix heap buffer overflow in fgetwln()

In the function fgetwln() there's a 4 byte heap overflow.

There is a while loop that has this check to see whether there's still
enough space in the buffer:

if (!fb->len || wused > fb->len) {

If this is true more memory gets allocated. However this test won't be
true if wused == fb->len, but at that point wused already points out
of the buffer. Some lines later there's a write to the buffer:

fb->wbuf[wused++] = wc;

This bug was found with the help of address sanitizer.

Warned-by: ASAN
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93881
Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Add missing <sys/stat.h> include

The test in test/strmode.c can fail to compile depending on the
optimization flags used.

The constants that are used in this file (S_IFREG etc.) come from the
<sys/stat.h> include file. It seems gcc ignores this error if one
compiles with "-O2" (default), but if one uses no optimization it fails.

Add the missing include and it works all the time.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=93880
Signed-off-by: Guillem Jover <guillem@hadrons.org>

test: Fix success return code for arc4random unit test

test: Fix race condition in headers-*.sh

When running tests in parallel (e.g. using `make -j4 check`), the header
tests currently fail due to headers-overlay.sh and headers-system.sh
both generating headers-gen.c simultaneously, resulting in garbled
output. Fix this by using separate C files for the tests.

Signed-off-by: Lukas Fleischer <lfleischer@lfos.de>
Signed-off-by: Guillem Jover <guillem@hadrons.org>

Release libbsd 0.8.1

Add support for GNU/Hurd to getentropy()

Reuse the getentropy code for Linux on the Hurd, which has fallbacks
for when the better interfaces are not present. And remove all the code
that is not supported currently on the Hurd. Ideally the Hurd should
get an equivalent interfaces that does not suffer from the same
problems as /dev/urandom.

test: Add new unit tests for individual headers usage

Turn <bsd/bsd.h> inert when using LIBBSD_OVERLAY

Also print a warning stating this fact.

Add missing include to <md5.h>

The header was not self-contained, it was missing definitions for some
types included in <sys/types.h>.

Use the non-overlayed libbsd headers when we need our own definitions

Switch COPYING to Debian copyright machine readable format 1.0

Relicense my contribution to BSD-2-clause

This avoids having two licenses on the same file.

Add missing copyright and license headers

Fix getentropy implementation to use the correct system hooks

Include getentropy_<SYSTEM>.c instead of arc4random_<SYSTEM>.c.

Add support for GNU/kFreeBSD for closefrom() and getentropy()

Unify most arc4random Unix hooks into a single file

The Unix hook should work for most Unix-like systems, move glibc
specific code there and a FreeBSd specific comment, and remove the rest.
Also change the code to always fallback to use the generic Unix code.

This should cover GNU/Hurd and GNU/kFreeBSD among others.

Use local SHA512 header

Actually use the local private SHA512 header instead of relying on the
OpenSSL one for no good reason. Add definition for expected macro
SHA512_DIGEST_LENGTH.

Release libbsd 0.8.0

Update license and copyright information

Update arc4random module from OpenBSD and LibreSSL

Rework arc4random_stir() and arc4random_addrandom() code over the new
internal API, and documentation in the man page. Adapt the code to the
local build system.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=85827

Add private getentropy module from OpenBSD and LibreSSL

Adapt the code to the local build system.

Add private SHA512 module from FreeBSD

Add NIOS2 support to nlist()

Add support for the NIOS2 soft-core CPU provided by Altera.

Signed-off-by: Marek Vasut <marex@denx.de>
Signed-off-by: Guillem Jover <guillem@hadrons.org>
Cc: Ley Foon Tan <lftan@altera.com>
Cc: Thomas Chou <thomas@wytron.com.tw>
Cc: Walter Goossens <waltergoossens@home.nl>

man: Rename funopen.3 to funopen.3bsd to avoid clash with funtools

The funtools project ships a man page with the same, name. And although
it mith probably make more sense to rename the man page there, as BSD
systems will certainly not do so, this is the easiest and fastest way
to avoid a file conflict.

test: Check asprintf() return code

test: Add unit test for strmode()

test: Add unit test for arc4random()

build: Make git log invocation immune to local configuration

build: Move hash/helper.c into new libbsd_la_included_sources

Use this variable in EXTRA_DIST and libbsd_la_DEPENDENCIES.

build: Move proctitle_LDFLAGS inside BUILD_LIBBSD_CTOR conditional

Add explicit_bzero() function from OpenBSD

Make closefrom_procfs() fail when reallocarray() fails

Lock the file streams in fgetln() and fparseln()

The fparseln() function had the NetBSD uppercase macros stubbed out,
so replace them with the actual stdio ones. The fgetln() function was
missing any locking at all.

Mark functions handling format strings with __printflike

Add compile and link-time deprecation warnings for fgetln()

Although the current implementation in libbsd is probably one of the
safest ones around, it still poses some problems when used with many
file streams. This function has now a replacement, that is both more
standard and portable. Ask users to switch to getline(3) instead.

build: Add support for linker warnings

Switch fparseln() implementation from fgetln() to getline()

test: Add fparseln() unit test

test: Refactor stream testing functions into a new module

Avoid left shift overflow in reallocarray

Some 64-bit platforms (e.g. Windows 64) have a 32-bit long. So, shifting
1UL 32-bits to the left causes an overflow. This replaces the constant
1UL with (size_t)1 so that we get the correct constant size for the
platform.

Import from OpenBSD.

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

Make mergesort setup() static

Use ANSI C prototypes

Use reallocarray() instead of malloc() or realloc()

Do not close file descriptors while scanning the /proc filesystem

Closing file descriptors changes the content of the fd directories in
the /proc filesystem, which means readdir() might get very confused.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=85663

Move procfs based implementation into a new closefrom_procfs() function

Refactor file descriptor closure into a new closefrom_close()

Update closefrom() function

Import from sudo. Adapt the build system to detect the required features.

test: Add closefrom() unit test

test: Add test case for fpurge(NULL)

test: Move and activate fpurge() test case from module to a dedicated file

build: Centralize testsuite LDADD setting in a single variable

Sync queue(3) from FreeBSD

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=85147

Add __offsetof, __rangeof and __containerof to sys/cdefs.h

Import and adapt from FreeBSD.

Add __DECONST, __DEVOLATILE and __DEQUALIFY macros to sys/cdefs.h

Import from FreeBSD.

Add new man page for reallocarray(3)

Import man page from OpenBSD.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=85622

Use stdint integer types instead of BSD legacy ones

Fix arc4random() and arc4random_stir() prototypes

These two functions accept no arguments.  The prototypes should reflect
this.  This change lets the compiler warn about certain (admittedly
silly) mistakes.

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

Release libbsd 0.7.0

build: Remove hard requirement for GNU .init_array section support

In case the support is not available, just stop building the
libbsd-ctor.a library, which is a nice to have thing, but should not
have been a hard requirement from the start. This should allow to
build libbsd on non-glibc based systems using another libc.

Add reallocarray() function from OpenBSD

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

Add stringlist module from NetBSD

Add getbsize() function

Import code from DragonFlyBSD and man page from FreeBSD.

Add OpenRISC support to nlist()

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

Add timeconv module from FreeBSD

Inline license information from FreeBSD root dir COPYRIGHT file.

Add funopen() function

This is a wrapper over the glibc fopencookie() function.

We diverge from the FreeBSD, OpenBSD and DragonFlyBSD declarations,
because seekfn() there wrongly uses fpos_t, assuming it's an integral
type, and any code using that on a system where fpos_t is a struct
(such as GNU-based systems or NetBSD) will fail to build. In which case,
as the code has to be modified anyway, we might just as well use the
correct declaration.

Handle glibc partial header inclusions

The glibc headers use selective inclusions through the __need_NAME
mechanism to avoid circular dependencies.

The problem is that if we are being overlaid, and have been requested
a partial inclusion, when we pass control to the system header, then
we might miss definitions needed by our own header, resulting in build
failures.

Workaround that by catching current partial requests, and skip the
current inclusion.

build: Set subdir-objects automake option

Bump automake minimal version to 1.9.

Fix dehumanize_number() to correctly detect overflows

Do not allow numbers greated than INT64_MAX and smaller than INT64_MIN.
Clarify the positive sign value by prefixing it with an explicit +.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=66909

Release libbsd 0.6.0

Warn when setproctitle() gets called before initialization

Try to give a helpful message in case the program is not initializing
the setproctitle() machinery.

Move setproctitle() automatic initialization to its own library

The automatic initialization cannot be part of the main shared library,
because there is no thread-safe way to change the environ global
variable. This is not a problem if the initializaion happens just at
program load time, but becomes one if the shared library is directly or
indirectly dlopen()ed during the execution of the program, which could
have either kept references to the old environ or could change it in
some other thread. This has been observed for example on systems using
Samba NSS modules.

To avoid any other possible fallout, the constructor is split into a
new static library that needs to be linked explicitly into programs
using setproctitle(). As an additional safety measure the pkg-config
linker flags will mark the program as not allowing to be dlopen()ed
so that we avoid the problem described above.

Reported-by: Jan Alexander Steffens (heftig) <jan.steffens@gmail.com>
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=66679

build: Move version ABI from Makefile to configure.ac

It's easier to find there, and the value can be reused in case we have
to provide another shared library.

Release libbsd 0.5.2

Create a shallow copy of environ before replacing it in setproctitle()

Because clearenv() or setenv() might free the environ array of pointers,
we should make sure to copy it so that we can access it later on when
doing the deep copy via setenv().

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=65470

Specify setproctitle_stub() signature manually if typeof is missing

Do not stop exporting the function in the version node even if typeof
is not available, as that would break ABI.

Try to check if setproctitle() constructor got passed arguments

Force setproctitle() into .init_array section

The GNU .init_array support is an extension over the standard System V
ABI .init_array support, which passes the main() arguments to the init
function.

This support comes in three parts. First the dynamic linker (from glibc)
needs to support it. Then function pointers need to be placed in the
section, for example by using __attribute__((constructor)), that the
compiler (gcc or clang for example) might place in section .ctors and
the linker (from binutils) will move to .init_array on the output
object, or by placing them directly into .init_array by the compiler
when compiling. If this does not happen and the function pointers end
up in .ctors, then they will not get passed the main() arguments, which
we do really need in this case.

But this relies on recent binutils or gcc having native .init_array
support, and not having it disabled through --disable-initfini-array.

To guarantee we get the correct behaviour, let's just place the function
pointer in the .init_array section directly, so we only require a recent
enough glibc.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=65029

test: Try setting and getting an environment variable after setproctitle()

build: Ignore automake 1.13+ test suite generated files

Release libbsd 0.5.1

test: Mark a literal integer as long long

This fixes build failures on 32-bit architectures.

Release libbsd 0.5.0

build: Handle the ChangeLog as a distributed-only file

The ChangeLog file is distributed, and cannot be regenerated outside
of the git repository, so do not remove it in DISTCLEANFILES, and move
the generation code into dist-hook, which also avoids unnecessary
computation during normal builds.

build: Use AM_V_at for mkdir

build: Compress the distribution tarball with xz instead of gzip

Fix comparison between signed and unsigned integer compiler warnings

Update radixsort module from NetBSD

Merge some interesting changes.

Update setmode module from NetBSD

Merge some interesting changes.

Fix getpeereid() compilation on Solaris

The code in getpeereid() is unlikely to compile as ucred_t is an opaque
struct (ucred_t * works but ucred_t does not). Either you need to give
a pointer initialized to NULL and getpeerucred() allocates a new ucred
or you call it with an allocated ucred as in this patch.

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

Switch dehumanize_number() to use expand_number()

The function is a duplicate of expand_number(), but covering less
prefixes and with a slightly different function signature.

Spotted-by: Peter da Silva <resuna@gmail.com>

test: Add new humanize unit test

test: Add setproctitle() unit test

Make setproctitle() available in 0.2 and 0.5 version nodes

Make the 0.5 version the default, so that code wanting the actual
implemented version can get a proper versioned depdendency. For code
linked against the old version, make it available as an alias.

Implement sendmail semantics for setproctitle()

Prefix the title with "progname: ", and skip it if the format string
starts with '-' (which gets skipped on output too).