Address andy's feedback

ipcc/src/ffi.rs

@@ -42,7 +42,10 @@ pub(crate) const LIBIPCC_ERR_KEY_VALTOOLONG: libipcc_err_t = 7;
 pub(crate) const LIBIPCC_ERR_KEY_ZERR: libipcc_err_t = 8;
 pub(crate) type libipcc_err_t = c_uint;
 
-pub type libipcc_key_flag_t = ::std::os::raw::c_uint;
+/// Maxium length of an error message retrieved by libipcc_errmsg().
+pub(crate) const LIBIPCC_ERR_LEN: usize = 1024;
+
+pub(crate) type libipcc_key_flag_t = ::std::os::raw::c_uint;
 
 #[link(name = "ipcc")]
 extern "C" {

ipcc/src/handle.rs

@@ -31,8 +31,7 @@ fn ipcc_fatal_error<C: Into<String>>(
     let syserr = if syserr == 0 {
         "no system errno".to_string()
     } else {
-        let syserr = unsafe { libc::strerror(syserr) };
-        unsafe { CStr::from_ptr(syserr) }.to_string_lossy().into_owned()
+        std::io::Error::from_raw_os_error(syserr).to_string()
     };
     let inner = IpccErrorInner {
         context,
@@ -56,7 +55,7 @@ fn ipcc_fatal_error<C: Into<String>>(
 impl IpccHandle {
     pub fn new() -> Result<Self, IpccError> {
         let mut ipcc_handle: *mut libipcc_handle_t = ptr::null_mut();
-        let errmsg = CString::new(vec![1; 1024]).unwrap();
+        let errmsg = CString::new(vec![1; LIBIPCC_ERR_LEN]).unwrap();
         let errmsg_len = errmsg.as_bytes().len();
         let errmsg_ptr = errmsg.into_raw();
         let mut lerr = LIBIPCC_ERR_OK;

ipcc/src/lib.rs

@@ -173,7 +173,7 @@ pub struct IpccErrorInner {
     pub syserr: String,
 }
 
-/// These are the IPCC keys we can lookup.
+/// These are the IPCC keys we can look up.
 /// NB: These keys match the definitions found in libipcc (RFD 316) and should
 /// match the values in `[ipcc::Key]` one-to-one.
 #[derive(Debug, Clone, Copy)]