search

1wilkens / pam

Safe Rust API to the Linux Pluggable Authentication Modules (PAM)
https://docs.rs/pam/
Apache License 2.0
88 stars 36 forks source link

Portably refer to C char type #36

Closed Goorzhel closed 1 year ago

Goorzhel commented 2 years ago

Assuming it's i8 will work on Intel, but not on ARM, PowerPC, and so on.

https://doc.rust-lang.org/1.63.0/core/ffi/struct.CStr.html#examples https://doc.rust-lang.org/1.63.0/src/core/ffi/mod.rs.html#104

Goorzhel commented 2 years ago

Ah, apologies for the presumption, you're way ahead of me:

❯ ag -c const.c.char
src/client.rs:1
src/module.rs:7
src/functions.rs:3
src/env.rs:3

It seems that client.rs just had a missed spot.

1wilkens commented 1 year ago

Hey sorry for the late reply! Yeah that seems like an oversight. Thanks for catching that!