Merge pull request #1811 from cfallin/fix-warning

Fix build warnings (errors on CI) due to mmap flag rename and deprecation.
This commit is contained in:
Chris Fallin
2020-06-03 13:26:36 -07:00
committed by GitHub
4 changed files with 27 additions and 39 deletions

52
Cargo.lock generated
View File

@@ -1,14 +1,5 @@
# This file is automatically @generated by Cargo. # This file is automatically @generated by Cargo.
# It is not intended for manual editing. # It is not intended for manual editing.
[[package]]
name = "addr2line"
version = "0.12.1"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "a49806b9dadc843c61e7c97e72490ad7f7220ae249012fbda9ad0609457c0543"
dependencies = [
"gimli 0.21.0",
]
[[package]] [[package]]
name = "ahash" name = "ahash"
version = "0.2.18" version = "0.2.18"
@@ -94,17 +85,26 @@ checksum = "f8aac770f1885fd7e387acedd76065302551364496e46b3dd00860b2f8359b9d"
[[package]] [[package]]
name = "backtrace" name = "backtrace"
version = "0.3.48" version = "0.3.46"
source = "registry+https://github.com/rust-lang/crates.io-index" source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "0df2f85c8a2abbe3b7d7e748052fdd9b76a0458fdeb16ad4223f5eca78c7c130" checksum = "b1e692897359247cc6bb902933361652380af0f1b7651ae5c5013407f30e109e"
dependencies = [ dependencies = [
"addr2line", "backtrace-sys",
"cfg-if", "cfg-if",
"libc", "libc",
"object 0.19.0",
"rustc-demangle", "rustc-demangle",
] ]
[[package]]
name = "backtrace-sys"
version = "0.1.37"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "18fbebbe1c9d1f383a9cc7e8ccdb471b91c8d024ee9c2ca5b5346121fe8b4399"
dependencies = [
"cc",
"libc",
]
[[package]] [[package]]
name = "base64" name = "base64"
version = "0.11.0" version = "0.11.0"
@@ -362,7 +362,7 @@ dependencies = [
"cranelift-codegen-meta", "cranelift-codegen-meta",
"cranelift-codegen-shared", "cranelift-codegen-shared",
"cranelift-entity", "cranelift-entity",
"gimli 0.20.0", "gimli",
"hashbrown 0.7.2", "hashbrown 0.7.2",
"log", "log",
"peepmatic", "peepmatic",
@@ -418,7 +418,7 @@ dependencies = [
"cranelift-reader", "cranelift-reader",
"file-per-thread-logger", "file-per-thread-logger",
"filecheck", "filecheck",
"gimli 0.20.0", "gimli",
"log", "log",
"memmap", "memmap",
"num_cpus", "num_cpus",
@@ -480,7 +480,7 @@ version = "0.64.0"
dependencies = [ dependencies = [
"cranelift-codegen", "cranelift-codegen",
"cranelift-module", "cranelift-module",
"object 0.18.0", "object",
"target-lexicon", "target-lexicon",
] ]
@@ -924,12 +924,6 @@ dependencies = [
"stable_deref_trait", "stable_deref_trait",
] ]
[[package]]
name = "gimli"
version = "0.21.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "bcc8e0c9bce37868955864dbecd2b1ab2bdf967e6f28066d65aaac620444b65c"
[[package]] [[package]]
name = "glob" name = "glob"
version = "0.3.0" version = "0.3.0"
@@ -1211,12 +1205,6 @@ dependencies = [
"target-lexicon", "target-lexicon",
] ]
[[package]]
name = "object"
version = "0.19.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "9cbca9424c482ee628fa549d9c812e2cd22f1180b9222c9200fdfa6eb31aecb2"
[[package]] [[package]]
name = "once_cell" name = "once_cell"
version = "1.4.0" version = "1.4.0"
@@ -2257,7 +2245,7 @@ version = "0.17.0"
dependencies = [ dependencies = [
"anyhow", "anyhow",
"faerie", "faerie",
"gimli 0.20.0", "gimli",
"more-asserts", "more-asserts",
"target-lexicon", "target-lexicon",
"thiserror", "thiserror",
@@ -2339,7 +2327,7 @@ dependencies = [
"cranelift-frontend", "cranelift-frontend",
"cranelift-native", "cranelift-native",
"cranelift-wasm", "cranelift-wasm",
"gimli 0.20.0", "gimli",
"log", "log",
"more-asserts", "more-asserts",
"region", "region",
@@ -2369,11 +2357,11 @@ version = "0.17.0"
dependencies = [ dependencies = [
"anyhow", "anyhow",
"cfg-if", "cfg-if",
"gimli 0.20.0", "gimli",
"ittapi-rs", "ittapi-rs",
"lazy_static", "lazy_static",
"libc", "libc",
"object 0.18.0", "object",
"scroll", "scroll",
"serde", "serde",
"target-lexicon", "target-lexicon",

View File

@@ -111,7 +111,7 @@ impl Drop for PtrLen {
fn drop(&mut self) { fn drop(&mut self) {
if !self.ptr.is_null() { if !self.ptr.is_null() {
unsafe { unsafe {
region::protect(self.ptr, self.len, region::Protection::ReadWrite) region::protect(self.ptr, self.len, region::Protection::READ_WRITE)
.expect("unable to unprotect memory"); .expect("unable to unprotect memory");
libc::free(self.ptr as _); libc::free(self.ptr as _);
} }
@@ -179,7 +179,7 @@ impl Memory {
for &PtrLen { ref map, ptr, len } in &self.allocations[self.executable..] { for &PtrLen { ref map, ptr, len } in &self.allocations[self.executable..] {
if len != 0 && map.is_some() { if len != 0 && map.is_some() {
unsafe { unsafe {
region::protect(ptr, len, region::Protection::ReadExecute) region::protect(ptr, len, region::Protection::READ_EXECUTE)
.expect("unable to make memory readable+executable"); .expect("unable to make memory readable+executable");
} }
} }
@@ -191,7 +191,7 @@ impl Memory {
for &PtrLen { ptr, len } in &self.allocations[self.executable..] { for &PtrLen { ptr, len } in &self.allocations[self.executable..] {
if len != 0 { if len != 0 {
unsafe { unsafe {
region::protect(ptr, len, region::Protection::ReadExecute) region::protect(ptr, len, region::Protection::READ_EXECUTE)
.expect("unable to make memory readable+executable"); .expect("unable to make memory readable+executable");
} }
} }
@@ -208,7 +208,7 @@ impl Memory {
for &PtrLen { ref map, ptr, len } in &self.allocations[self.executable..] { for &PtrLen { ref map, ptr, len } in &self.allocations[self.executable..] {
if len != 0 && map.is_some() { if len != 0 && map.is_some() {
unsafe { unsafe {
region::protect(ptr, len, region::Protection::Read) region::protect(ptr, len, region::Protection::READ)
.expect("unable to make memory readonly"); .expect("unable to make memory readonly");
} }
} }
@@ -220,7 +220,7 @@ impl Memory {
for &PtrLen { ptr, len } in &self.allocations[self.executable..] { for &PtrLen { ptr, len } in &self.allocations[self.executable..] {
if len != 0 { if len != 0 {
unsafe { unsafe {
region::protect(ptr, len, region::Protection::Read) region::protect(ptr, len, region::Protection::READ)
.expect("unable to make memory readonly"); .expect("unable to make memory readonly");
} }
} }

View File

@@ -120,7 +120,7 @@ impl CodeMemory {
if !m.is_empty() { if !m.is_empty() {
unsafe { unsafe {
region::protect(m.as_mut_ptr(), m.len(), region::Protection::ReadExecute) region::protect(m.as_mut_ptr(), m.len(), region::Protection::READ_EXECUTE)
} }
.expect("unable to make memory readonly and executable"); .expect("unable to make memory readonly and executable");
} }

View File

@@ -182,7 +182,7 @@ impl Mmap {
// Commit the accessible size. // Commit the accessible size.
let ptr = self.ptr as *const u8; let ptr = self.ptr as *const u8;
unsafe { region::protect(ptr.add(start), len, region::Protection::ReadWrite) } unsafe { region::protect(ptr.add(start), len, region::Protection::READ_WRITE) }
.map_err(|e| e.to_string()) .map_err(|e| e.to_string())
} }