Missing safety blocks in solid/io.rs

[JorgeCepeda]

Location

• Second impl block for BorrowedFd<'_>, safety is in the first one:

  rust/library/std/src/os/solid/io.rs

  Lines 125 to 126 in dee7d0e

  	let fd = sys::net::cvt(unsafe { sys::net::netc::dup(self.as_raw_fd()) })?;
  	Ok(unsafe { OwnedFd::from_raw_fd(fd) })

• Near impl safety block of OwnedFd:

  rust/library/std/src/os/solid/io.rs

  Lines 170 to 171 in dee7d0e

  	fn drop(&mut self) {
  	unsafe { sys::net::netc::close(self.fd.as_inner()) };

Summary

This is my first issue so maybe there is no issue.
I'm checking some unsafe blocks just in case something is missing and there are those without safety blocks.

Since I saw a different file with a safety block in the drop function (see:

rust/library/core/src/cell/lazy.rs

Lines 150 to 152 in dee7d0e

	fn drop(&mut self) {
	// SAFETY: Invariant states it is valid, and we don't drop the old value.
	unsafe {

) I thought it may be missing here.

What guide could I check to make sure?
Thanks in advance.

[Noratrieb]

there are many underdocumented unsafe blocks in the standard library, especially in std::sys. PRs welcome