Skip to content

utils.c: fix parsing of addr:size and addr:size:path arguments - #17

Merged
xingrz merged 2 commits into
LISTENAI:masterfrom
sp5wwp:master
Aug 7, 2026
Merged

utils.c: fix parsing of addr:size and addr:size:path arguments#17
xingrz merged 2 commits into
LISTENAI:masterfrom
sp5wwp:master

Conversation

@sp5wwp

@sp5wwp sp5wwp commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

sp5wwp and others added 2 commits August 4, 2026 19:52
Keep strict integer validation and parse addr:size[:path] directly at
the delimiters. Avoid fixed path buffers, and reject empty paths and
invalid output pointers.

Treat only values with a 0x/0X prefix as hexadecimal. Parse all other
values as decimal.

Add CTest coverage for the C62 backup form, long and Windows paths,
overflow, empty fields, and trailing garbage. Run tests in Linux x64 CI.

Co-Authored-By: OpenAI Codex (GPT-5) <noreply@openai.com>
Signed-off-by: xychen <xychen@listenai.com>
@xingrz
xingrz merged commit 717e27a into LISTENAI:master Aug 7, 2026
10 checks passed
@xingrz

xingrz commented Aug 7, 2026

Copy link
Copy Markdown

@sp5wwp Thanks for the contribution. We’re impressed by how deeply you and the OpenRTX team have explored the CSK6. BD4VOW has already contacted us about the recording sample-rate issue, and we expect to provide DSP firmware with 48 kHz recording support sometime next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants