Fork 1

croil self-assigned this 2026年03月28日 00:48:21 +01:00

croil added 4 commits 2026年03月28日 00:48:22 +01:00

This patch fixes non-compliance with strict C99 by changing the
`ssize_t` type to `intmax_t`, implementing `_strnlen_i()` using
`memchr()` as a replacement for `strnlen()`, which is an extension, and
fixes an edge case where if the first path passed to any of the
`crpath*cat()` family is trivial, `nacc` will be set to an uninitialized
value, causing undefined behavior.

- Significant enhancements to documentation in header file.
 - Updated argument checking so that we always ensure that the real length of
 any `path` argument is clamped to the first terminating zero found in the
 buffer before `np`.
 - Updated `_pathlast_i()` so that nothing will be written to `buf` or `*len`
 assuming they are provided until the last item is found.
 - Took advantage of the updated argument checks to eliminate some redundant
 internal calls to `_strnlen_i()`.
 - Implemented `crpathup()` for fetching the semantic parent of a path.
 - Added `crpatherrs()` utility for getting string descriptions of error codes.
 - Updated `example.c` to check the result of `crpathup()` and to be more
 verbose for pedagogical reasons.

croil added 1 commit 2026年03月28日 00:57:33 +01:00

croil referenced this pull request from a commit 2026年03月28日 01:00:46 +01:00

croil merged commit d9fdd4b881 into main 2026年03月28日 01:01:19 +01:00

croil deleted branch staging 2026年03月28日 01:01:20 +01:00

Sign in to join this conversation.

Clear labels

Compat/Breaking

Breaking change that won't be backward compatible

Kind/Bug

Something is not working

Kind/Documentation

Documentation changes

Kind/Enhancement

Improve existing functionality

Kind/Feature

New functionality

Kind/Security

This is security issue

Kind/Testing

Issue or pull request related to testing

Priority

Critical

The priority is critical

Priority

High

The priority is high

Priority

Low

The priority is low

Priority

Medium

The priority is medium

Resolved

Marks the issue as resolved.

Reviewed

Confirmed

Issue has been confirmed

Reviewed

Duplicate

This issue or pull request already exists

Reviewed

Invalid

Invalid issue

Reviewed

Won't Fix

This issue won't be fixed

Status

Abandoned

Somebody has started to work on this but abandoned work

Status

Blocked

Something is blocking this issue or pull request

Status

Need More Info

Feedback is required to reproduce issue or to continue work

Milestone

No milestone

Subscribe

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".