rust-lang / libc

Raw bindings to platform APIs for Rust
https://docs.rs/libc
Apache License 2.0
2.07k stars 1.04k forks source link

`ifa_flags` is `uint_64` on Solaris/illumos #3726

Closed mtelka closed 3 months ago

mtelka commented 4 months ago

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.

devnexen commented 4 months ago

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

mtelka commented 4 months ago

Why not main first?

devnexen commented 4 months ago

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).