Skip to content

Implement SIGPIPE delivery on EPIPE#791

Open
CvvT wants to merge 2 commits into
mainfrom
weiteng/handle_epipe
Open

Implement SIGPIPE delivery on EPIPE#791
CvvT wants to merge 2 commits into
mainfrom
weiteng/handle_epipe

Conversation

@CvvT
Copy link
Copy Markdown
Contributor

@CvvT CvvT commented Apr 21, 2026

handle EPIPE error for various syscalls like write and sendto.

@CvvT CvvT marked this pull request as ready for review April 21, 2026 18:21
Comment thread litebox_shim_linux/src/syscalls/file.rs Outdated
use litebox_platform_multiplex::Platform;

use crate::{ConstPtr, GlobalState, MutPtr, ShimFS, Task};
use crate::{ConstPtr, GlobalState, MutPtr, ShimFS, Task, syscalls::signal::siginfo_kill};
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By convention, use declarations for types/macros/modules are fine, but we make sure not to do so for functions, in order to match the conventions most mature Rust projects use.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. Fixed now, thanks!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

🤖 SemverChecks 🤖 No breaking API changes detected

Note: this does not mean API is unchanged, or even that there are no breaking changes; simply, none of the detections triggered.

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