Skip to content

cpu/native/fs: fix stale errno reading in mount()#22252

Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom
fabian18:pr/native_fs_mount_stale_errno
May 8, 2026
Merged

cpu/native/fs: fix stale errno reading in mount()#22252
crasbe merged 1 commit intoRIOT-OS:masterfrom
fabian18:pr/native_fs_mount_stale_errno

Conversation

@fabian18
Copy link
Copy Markdown
Contributor

@fabian18 fabian18 commented May 7, 2026

Contribution description

On native _mount, errno is read also success case and value is falsely interpreted as an error.

Testing procedure

BOARD=native make -C tests/sys/shell_scripting

nvm0 folder is created but test fails because errno is falsely interpreted.
On second attempt, the test succeeds because nvm0 already was created.

Issues/PRs references

Issue noted in CI of #22158

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • none

@github-actions github-actions Bot added Platform: native Platform: This PR/issue effects the native platform Area: cpu Area: CPU/MCU ports labels May 7, 2026
@fabian18 fabian18 added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label May 7, 2026
@crasbe crasbe added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 7, 2026
@riot-ci
Copy link
Copy Markdown

riot-ci commented May 7, 2026

Murdock results

✔️ PASSED

93b8170 cpu/native/fs: fix stale errno reading in mount()

Success Failures Total Runtime
11107 0 11108 10m:58s

Artifacts

@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented May 7, 2026

Please rebase, now that #22251 is merged :)

@crasbe crasbe added the State: needs rebase State: The codebase was changed since the creation of the PR, making a rebase necessary label May 7, 2026
@fabian18 fabian18 force-pushed the pr/native_fs_mount_stale_errno branch from 552974e to 93b8170 Compare May 8, 2026 08:06
@crasbe crasbe removed the State: needs rebase State: The codebase was changed since the creation of the PR, making a rebase necessary label May 8, 2026
@crasbe crasbe enabled auto-merge May 8, 2026 08:36
@crasbe crasbe added this pull request to the merge queue May 8, 2026
Merged via the queue into RIOT-OS:master with commit d0be7c6 May 8, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: native Platform: This PR/issue effects the native platform Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants