hongshaorou/rustdesk
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove SERVER_CLIPBOARD_ENABLED unsafe code, by using AtomicBool

Browse Source
This commit is contained in:
Saverio Miroddi 2022-05-24 18:02:33 +02:00
parent 7f50fe3ea0
commit c7f452752d
1 changed files with 9 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/ui/remote.rs
View File

@ -70,7 +70,7 @@ static mut IS_IN: bool = false;
static KEYBOARD_HOOKED: AtomicBool = AtomicBool::new(false); static KEYBOARD_HOOKED: AtomicBool = AtomicBool::new(false);
static mut SERVER_KEYBOARD_ENABLED: bool = true; static mut SERVER_KEYBOARD_ENABLED: bool = true;
static mut SERVER_FILE_TRANSFER_ENABLED: bool = true; static mut SERVER_FILE_TRANSFER_ENABLED: bool = true;
static mut SERVER_CLIPBOARD_ENABLED: bool = true; static SERVER_CLIPBOARD_ENABLED: AtomicBool = AtomicBool::new(true);
#[cfg(windows)] #[cfg(windows)]
static mut IS_ALT_GR: bool = false; static mut IS_ALT_GR: bool = false;
@ -1382,7 +1382,7 @@ impl Remote {
Ok((mut peer, direct)) => { Ok((mut peer, direct)) => {
unsafe { unsafe {
SERVER_KEYBOARD_ENABLED = true; SERVER_KEYBOARD_ENABLED = true;
SERVER_CLIPBOARD_ENABLED = true; SERVER_CLIPBOARD_ENABLED.store(true, Ordering::SeqCst);
SERVER_FILE_TRANSFER_ENABLED = true; SERVER_FILE_TRANSFER_ENABLED = true;
} }
self.handler self.handler
@ -1464,7 +1464,7 @@ impl Remote {
} }
unsafe { unsafe {
SERVER_KEYBOARD_ENABLED = false; SERVER_KEYBOARD_ENABLED = false;
SERVER_CLIPBOARD_ENABLED = false; SERVER_CLIPBOARD_ENABLED.store(false, Ordering::SeqCst);
SERVER_FILE_TRANSFER_ENABLED = false; SERVER_FILE_TRANSFER_ENABLED = false;
} }
} }
@ -1518,7 +1518,7 @@ impl Remote {
} }
_ => {} _ => {}
} }
if !unsafe { SERVER_CLIPBOARD_ENABLED } if !SERVER_CLIPBOARD_ENABLED.load(Ordering::SeqCst)
|| !unsafe { SERVER_KEYBOARD_ENABLED } || !unsafe { SERVER_KEYBOARD_ENABLED }
|| lc.read().unwrap().disable_clipboard || lc.read().unwrap().disable_clipboard
{ {
@ -1978,7 +1978,7 @@ impl Remote {
self.check_clipboard_file_context(); self.check_clipboard_file_context();
if !(self.handler.is_file_transfer() if !(self.handler.is_file_transfer()
|| self.handler.is_port_forward() || self.handler.is_port_forward()
|| !unsafe { SERVER_CLIPBOARD_ENABLED } || !SERVER_CLIPBOARD_ENABLED.load(Ordering::SeqCst)
|| !unsafe { SERVER_KEYBOARD_ENABLED } || !unsafe { SERVER_KEYBOARD_ENABLED }
|| self.handler.lc.read().unwrap().disable_clipboard) || self.handler.lc.read().unwrap().disable_clipboard)
{ {
@ -2188,9 +2188,7 @@ impl Remote {
.call2("setPermission", &make_args!("keyboard", p.enabled)); .call2("setPermission", &make_args!("keyboard", p.enabled));
} }
Permission::Clipboard => { Permission::Clipboard => {
unsafe { SERVER_CLIPBOARD_ENABLED.store(p.enabled, Ordering::SeqCst);
SERVER_CLIPBOARD_ENABLED = p.enabled;
}
self.handler self.handler
.call2("setPermission", &make_args!("clipboard", p.enabled)); .call2("setPermission", &make_args!("clipboard", p.enabled));
} }