mirrors/gst-plugins-rs
mirrors/gst-plugins-rs
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs.git synced 2024-06-23 00:20:34 +00:00
Code Issues Projects Releases Wiki Activity

ts-udpsrc: Creating a gio::Socket from a raw fd/socket is unsafe

Browse source 
So put it into an unsafe block.
This commit is contained in:
Sebastian Dröge 2018-12-03 13:02:35 +02:00
parent 9750195caa
commit 3026e56cfb
1 changed files with 8 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
gst-plugin-threadshare/src/udpsrc.rs
View file

@ -697,15 +697,13 @@ impl UdpSrc {
// Store the socket as used-socket in the settings
#[cfg(unix)]
{
unsafe {
use libc;
let fd = unsafe { libc::dup(socket.as_raw_fd()) };
let fd = libc::dup(socket.as_raw_fd());
// This is technically unsafe because it allows
// us to share the fd between the socket and the
// GIO socket below, but safety of this is the
// job of the application
// This is unsafe because it allows us to share the fd between the socket and the
// GIO socket below, but safety of this is the job of the application
struct FdConverter(RawFd);
impl IntoRawFd for FdConverter {
fn into_raw_fd(self) -> RawFd {
@ -725,13 +723,11 @@ impl UdpSrc {
self.settings.lock().unwrap().used_socket = Some(wrapper);
}
#[cfg(windows)]
{
let fd = unsafe { dup_socket(socket.as_raw_socket() as _) as _ };
unsafe {
let fd = dup_socket(socket.as_raw_socket() as _) as _;
// This is technically unsafe because it allows
// us to share the fd between the socket and the
// GIO socket below, but safety of this is the
// job of the application
// This is unsafe because it allows us to share the fd between the socket and the
// GIO socket below, but safety of this is the job of the application
struct SocketConverter(RawSocket);
impl IntoRawSocket for SocketConverter {
fn into_raw_socket(self) -> RawSocket {