diff options
author | Dmitry Torokhov <dtor@chromium.org> | 2018-02-28 14:32:24 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-03-01 16:13:07 -0800 |
commit | 505c6f98881e8998334d3f577334dd1570bca8d0 (patch) | |
tree | 4b1cc1aec64c57ab28d80d639218bb7270fe9095 /vm_control/src | |
parent | cb47da4910c846bae990ba04cf4d2dd94d0301ff (diff) | |
download | crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar.gz crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar.bz2 crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar.lz crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar.xz crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.tar.zst crosvm-505c6f98881e8998334d3f577334dd1570bca8d0.zip |
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>
Diffstat (limited to 'vm_control/src')
-rw-r--r-- | vm_control/src/lib.rs | 4 |
1 files changed, 2 insertions, 2 deletions
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 @@ impl VmResponse { 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 @@ mod tests { 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); |