commit 505c6f98881e8998334d3f577334dd1570bca8d0
author Dmitry Torokhov <dtor@chromium.org> Wed Feb 28 22:32:24 2018
committer chrome-bot <chrome-bot@chromium.org> Fri Mar 02 00:13:07 2018
tree 4b1cc1aec64c57ab28d80d639218bb7270fe9095
parent cb47da4910c846bae990ba04cf4d2dd94d0301ff

vm_control: more error signedness fixes

Fix a couple more spaces where we used negative errno values.

TEST=cargo test -p vm_control
BUG=None

Change-Id: Id622192d025d0ac733419411492ca53a50f957f3
Signed-off-by: Dmitry Torokhov <dtor@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/942087
Reviewed-by: Zach Reizner <zachr@chromium.org>

vm_control/src/lib.rs

diff --git a/vm_control/src/lib.rs b/vm_control/src/lib.rs
index 0f39818..ec0522e 100644
--- a/vm_control/src/lib.rs
+++ b/vm_control/src/lib.rs
@@ -288,7 +288,7 @@
         match resp.type_.into() {
             VM_RESPONSE_TYPE_OK => Ok(VmResponse::Ok),
             VM_RESPONSE_TYPE_ERR => {
-                Ok(VmResponse::Err(SysError::new(-(resp.errno.to_native() as i32))))
+                Ok(VmResponse::Err(SysError::new(resp.errno.to_native() as i32)))
             }
             VM_RESPONSE_TYPE_REGISTER_MEMORY => {
                 Ok(VmResponse::RegisterMemory {
@@ -440,7 +440,7 @@
     fn resp_err() {
         let (s1, s2) = UnixDatagram::pair().expect("failed to create socket pair");
         let mut scm = Scm::new(1);
-        let r1 = VmResponse::Err(SysError::new(-89));
+        let r1 = VmResponse::Err(SysError::new(libc::EDESTADDRREQ));
         r1.send(&mut scm, &s1).unwrap();
         let r2 = VmResponse::recv(&mut scm, &s2).unwrap();
         assert_eq!(r1, r2);