Maybe unsound in RLocal::new

Hello, thank you for your contribution in this project, I am scanning the unsoundness problem in rust project.
I notice the following code:

pub struct RLocal<T: Copy> {
    old_value: T,
    variable: *mut T,
}

impl<T> RLocal<T>
where
    T: Copy,
{
    pub fn new(new_value: T, variable: *mut T) -> RLocal<T> {
        unsafe {
            let old_value = libr::get(variable);
            libr::set(variable, new_value);

            Self {
                old_value,
                variable,
            }
        }
    }
}

Considering that pub mod raii, and new is also a pub function. I assume that users can directly call this function. This potential situation could result in libr::get being called to a null pointer, and might trigger undefined behavior (UB). For safety reasons, I felt it necessary to report this issue. If you have performed checks elsewhere that ensure this is safe, please don’t take offense at my raising this issue.
I suggest Several possible fixes:

1. If there is no external usage for RLocal or new, they should not marked as pub, at least its new should not marked as pub
2. new method should add additional check for null pointer.
3. mark new method as unsafe and proper doc to let users know that they should provide valid Pointers.

Bắt đầu bằng cách tìm module raii công khai và RLocal::new, sau đó kiểm tra các bên gọi và các giả định về con trỏ xung quanh libr::get và libr::set. Xác định xem API có thể nhận một con trỏ null hoặc một con trỏ không hợp lệ theo cách khác hay không; được xem là hoàn tất khi hợp đồng an toàn hoặc khả năng hiển thị đã rõ ràng và đường dẫn đến hành vi không xác định được báo cáo đã được xử lý.