Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ifa_flags is uint_64 on Solaris/illumos #3726

Closed
mtelka opened this issue May 28, 2024 · 3 comments · Fixed by #3729
Closed

ifa_flags is uint_64 on Solaris/illumos #3726

mtelka opened this issue May 28, 2024 · 3 comments · Fixed by #3729
Labels
C-bug Category: bug

Comments

@mtelka
Copy link
Contributor

mtelka commented May 28, 2024

The ifa_flags is declared as c_ulong here for solarish, but actually in both Solaris and illumos it is uint_64:

I think the type of ifa_flags should be changed to u64 here.

This issue was found during the illumos support fix work for GStreamer.

@mtelka mtelka added the C-bug Category: bug label May 28, 2024
@devnexen
Copy link
Contributor

If you feel like it, you can create a PR targeting libc-0.2 for starter.

@mtelka
Copy link
Contributor Author

mtelka commented May 29, 2024

Why not main first?

@devnexen
Copy link
Contributor

sure, but regardless your fix will need to be backported to libc-0.2 if we want to land for the next release (main relates to future 0.3).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants