Skip to content

pal_console.c: cast TIOCSWINSZ to int when HAVE_IOCTL_WITH_INT_REQUEST - #82676

Merged
stephentoub merged 1 commit into
dotnet:mainfrom
ayakael:musl/cast-pal-console-int
Feb 27, 2023
Merged

pal_console.c: cast TIOCSWINSZ to int when HAVE_IOCTL_WITH_INT_REQUEST#82676
stephentoub merged 1 commit into
dotnet:mainfrom
ayakael:musl/cast-pal-console-int

Conversation

@ayakael

@ayakaelayakael commented Feb 25, 2023•
edited
Loading

Copy link
Copy Markdown
Contributor

Adds missing cast to int when HAVE_IOCTL_WITH_INT_REQUEST

Related to #82173

Per @am11:

On some Unix-likes, such as linux with musl-libc and illumos, the second argument of ioctl is
int request instead of unsigned long request.

/cc: @Sapana-Khemkar

@ghostghost added the community-contribution Indicates that the PR has been added by a community member label Feb 25, 2023
@ayakaelayakael changed the title pal_console.c: cast TIOCSWINSZ to int when HAVE_IOCTL_WITH_INT_REQUESTpal_console.c: cast TIOCSWINSZ to int when HAVE_IOCTL_WITH_INT_REQUESTFeb 25, 2023
@stephentoub
stephentoub merged commit fb13c2c into dotnet:mainFeb 27, 2023
@ghostghost locked as resolved and limited conversation to collaborators Mar 29, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@stephentoubstephentoubstephentoub approved these changes

Assignees

No one assigned

Labels

community-contributionIndicates that the PR has been added by a community member

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ayakael@stephentoub