[INFO] cloning repository https://github.com/MrE-Fog/boringtun [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/MrE-Fog/boringtun" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FMrE-Fog%2Fboringtun", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FMrE-Fog%2Fboringtun'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] ac3718c6684b9b618126aa0b3e1f8d6250b45e69 [INFO] checking MrE-Fog/boringtun against try#b1f2594eac607c1f051534800237eeedb5590a49 for pr-139493-3 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FMrE-Fog%2Fboringtun" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'... [INFO] [stderr] done. [INFO] removed /workspace/builds/worker-6-tc2/source/.cargo/config [INFO] started tweaking git repo https://github.com/MrE-Fog/boringtun [INFO] finished tweaking git repo https://github.com/MrE-Fog/boringtun [INFO] tweaked toml for git repo https://github.com/MrE-Fog/boringtun written to /workspace/builds/worker-6-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/MrE-Fog/boringtun on toolchain b1f2594eac607c1f051534800237eeedb5590a49 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+b1f2594eac607c1f051534800237eeedb5590a49" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/MrE-Fog/boringtun already has a lockfile, it will not be regenerated [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+b1f2594eac607c1f051534800237eeedb5590a49" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: virtual workspace defaulting to `resolver = "1"` despite one or more workspace members being on edition 2021 which implies `resolver = "2"` [INFO] [stderr] | [INFO] [stderr] = note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest [INFO] [stderr] = note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest [INFO] [stderr] = note: for more details see https://doc.rust-lang.org/cargo/reference/resolver.html#resolver-versions [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+b1f2594eac607c1f051534800237eeedb5590a49" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] c619a2e819f55c46b6ac29b3e474cd5592ff426ac64d40de484b8c66e932b61f [INFO] running `Command { std: "docker" "start" "-a" "c619a2e819f55c46b6ac29b3e474cd5592ff426ac64d40de484b8c66e932b61f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c619a2e819f55c46b6ac29b3e474cd5592ff426ac64d40de484b8c66e932b61f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c619a2e819f55c46b6ac29b3e474cd5592ff426ac64d40de484b8c66e932b61f", kill_on_drop: false }` [INFO] [stdout] c619a2e819f55c46b6ac29b3e474cd5592ff426ac64d40de484b8c66e932b61f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:aa71247004a7fa38d13ec170f48f06cdedf5bc50b2a8645e56ed7e992e6fa513" "/opt/rustwide/cargo-home/bin/cargo" "+b1f2594eac607c1f051534800237eeedb5590a49" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 2523b51e7c742ce3e62d463c2d3ffd3e9edf95ce00850b52baa5f8da015f43a6 [INFO] running `Command { std: "docker" "start" "-a" "2523b51e7c742ce3e62d463c2d3ffd3e9edf95ce00850b52baa5f8da015f43a6", kill_on_drop: false }` [INFO] [stderr] warning: virtual workspace defaulting to `resolver = "1"` despite one or more workspace members being on edition 2021 which implies `resolver = "2"` [INFO] [stderr] | [INFO] [stderr] = note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest [INFO] [stderr] = note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest [INFO] [stderr] = note: for more details see https://doc.rust-lang.org/cargo/reference/resolver.html#resolver-versions [INFO] [stderr] Compiling libc v0.2.117 [INFO] [stderr] Compiling syn v1.0.86 [INFO] [stderr] Compiling parking_lot_core v0.9.1 [INFO] [stderr] Compiling cc v1.0.72 [INFO] [stderr] Checking ip_network_table-deps-treebitmap v0.5.0 [INFO] [stderr] Checking tracing-core v0.1.22 [INFO] [stderr] Checking thread_local v1.1.4 [INFO] [stderr] Checking sharded-slab v0.1.4 [INFO] [stderr] Checking ip_network v0.4.1 [INFO] [stderr] Compiling crossbeam-utils v0.8.7 [INFO] [stderr] Checking num_threads v0.1.3 [INFO] [stderr] Checking textwrap v0.11.0 [INFO] [stderr] Checking boxfnonce v0.1.1 [INFO] [stderr] Checking ip_network_table v0.2.0 [INFO] [stderr] Checking crossbeam-channel v0.5.2 [INFO] [stderr] Checking tracing-log v0.1.2 [INFO] [stderr] Checking tracing-subscriber v0.3.9 [INFO] [stderr] Compiling ring v0.16.20 [INFO] [stderr] Checking time v0.3.7 [INFO] [stderr] Checking atty v0.2.14 [INFO] [stderr] Checking daemonize v0.4.1 [INFO] [stderr] Checking clap v2.34.0 [INFO] [stderr] Checking parking_lot v0.12.0 [INFO] [stderr] Checking tracing-appender v0.2.1 [INFO] [stderr] Compiling synstructure v0.12.6 [INFO] [stderr] Compiling zeroize_derive v1.3.1 [INFO] [stderr] Compiling tracing-attributes v0.1.19 [INFO] [stderr] Checking zeroize v1.5.2 [INFO] [stderr] Checking tracing v0.1.31 [INFO] [stderr] Checking boringtun v0.4.0 (/opt/rustwide/workdir/boringtun) [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:99:12 [INFO] [stdout] | [INFO] [stdout] 99 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:120:12 [INFO] [stdout] | [INFO] [stdout] 120 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:141:12 [INFO] [stdout] | [INFO] [stdout] 141 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:178:12 [INFO] [stdout] | [INFO] [stdout] 178 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:231:12 [INFO] [stdout] | [INFO] [stdout] 231 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:99:12 [INFO] [stdout] | [INFO] [stdout] 99 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:120:12 [INFO] [stdout] | [INFO] [stdout] 120 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:141:12 [INFO] [stdout] | [INFO] [stdout] 141 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:178:12 [INFO] [stdout] | [INFO] [stdout] 178 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:231:12 [INFO] [stdout] | [INFO] [stdout] 231 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:17 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:38 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/device/allowed_ips.rs:59:17 [INFO] [stdout] | [INFO] [stdout] 59 | pub fn iter(&self) -> Iter { [INFO] [stdout] | ^^^^^ ^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 59 | pub fn iter(&self) -> Iter<'_, D> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/device/dev_lock.rs:28:17 [INFO] [stdout] | [INFO] [stdout] 28 | pub fn read(&self) -> LockReadGuard { [INFO] [stdout] | ^^^^^ ^^^^^^^^^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 28 | pub fn read(&self) -> LockReadGuard<'_, T> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `Box::::from_raw` that must be used [INFO] [stdout] --> boringtun/src/ffi/mod.rs:236:5 [INFO] [stdout] | [INFO] [stdout] 236 | Box::from_raw(tunnel); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: call `drop(Box::from_raw(ptr))` if you intend to drop the `Box` [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 236 | let _ = Box::from_raw(tunnel); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/noise/mod.rs:254:39 [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result { [INFO] [stdout] | ^^^^^ ^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result, WireGuardError> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking boringtun-cli v0.1.0 (/opt/rustwide/workdir/boringtun-cli) [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:99:12 [INFO] [stdout] | [INFO] [stdout] 99 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/poly1305.rs:120:12 [INFO] [stdout] | [INFO] [stdout] 120 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:141:12 [INFO] [stdout] | [INFO] [stdout] 141 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:178:12 [INFO] [stdout] | [INFO] [stdout] 178 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:231:12 [INFO] [stdout] | [INFO] [stdout] 231 | #[cfg_attr(feature = "cargo-clippy", allow(clippy::suspicious_arithmetic_impl))] [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ help: remove the condition [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = help: consider adding `cargo-clippy` as a feature in `Cargo.toml` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `ring::rand::SystemRandom` [INFO] [stdout] --> boringtun/src/crypto/mod.rs:14:9 [INFO] [stdout] | [INFO] [stdout] 14 | pub use ring::rand::SystemRandom; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:17 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:38 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/device/allowed_ips.rs:59:17 [INFO] [stdout] | [INFO] [stdout] 59 | pub fn iter(&self) -> Iter { [INFO] [stdout] | ^^^^^ ^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 59 | pub fn iter(&self) -> Iter<'_, D> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/device/dev_lock.rs:28:17 [INFO] [stdout] | [INFO] [stdout] 28 | pub fn read(&self) -> LockReadGuard { [INFO] [stdout] | ^^^^^ ^^^^^^^^^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 28 | pub fn read(&self) -> LockReadGuard<'_, T> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `Box::::from_raw` that must be used [INFO] [stdout] --> boringtun/src/ffi/mod.rs:236:5 [INFO] [stdout] | [INFO] [stdout] 236 | Box::from_raw(tunnel); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: call `drop(Box::from_raw(ptr))` if you intend to drop the `Box` [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 236 | let _ = Box::from_raw(tunnel); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/noise/mod.rs:254:39 [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result { [INFO] [stdout] | ^^^^^ ^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result, WireGuardError> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: methods `aead_init`, `seal`, and `open` are never used [INFO] [stdout] --> boringtun/src/crypto/chacha20poly1305/mod.rs:295:8 [INFO] [stdout] | [INFO] [stdout] 22 | impl ChaCha20Poly1305 { [INFO] [stdout] | --------------------- methods in this implementation [INFO] [stdout] ... [INFO] [stdout] 295 | fn aead_init(&self, nonce: &[u8]) -> [u32; 16] { [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 396 | pub fn seal(&self, nonce: &[u8], aad: &[u8], plaintext: &[u8], ciphertext: &mut [u8]) -> usize { [INFO] [stdout] | ^^^^ [INFO] [stdout] ... [INFO] [stdout] 405 | pub fn open<'a>( [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `IPV4_DST_IP_OFF` is never used [INFO] [stdout] --> boringtun/src/noise/mod.rs:31:7 [INFO] [stdout] | [INFO] [stdout] 31 | const IPV4_DST_IP_OFF: usize = 16; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: constant `IPV6_DST_IP_OFF` is never used [INFO] [stdout] --> boringtun/src/noise/mod.rs:37:7 [INFO] [stdout] | [INFO] [stdout] 37 | const IPV6_DST_IP_OFF: usize = 24; [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `1` is never read [INFO] [stdout] --> boringtun/src/noise/mod.rs:50:35 [INFO] [stdout] | [INFO] [stdout] 50 | WriteToTunnelV4(&'a mut [u8], Ipv4Addr), [INFO] [stdout] | --------------- ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `TunnResult` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 50 - WriteToTunnelV4(&'a mut [u8], Ipv4Addr), [INFO] [stdout] 50 + WriteToTunnelV4(&'a mut [u8], ()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `1` is never read [INFO] [stdout] --> boringtun/src/noise/mod.rs:51:35 [INFO] [stdout] | [INFO] [stdout] 51 | WriteToTunnelV6(&'a mut [u8], Ipv6Addr), [INFO] [stdout] | --------------- ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `TunnResult` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 51 - WriteToTunnelV6(&'a mut [u8], Ipv6Addr), [INFO] [stdout] 51 + WriteToTunnelV6(&'a mut [u8], ()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated items `set_static_private`, `dst_address`, and `is_expired` are never used [INFO] [stdout] --> boringtun/src/noise/mod.rs:162:12 [INFO] [stdout] | [INFO] [stdout] 122 | impl Tunn { [INFO] [stdout] | --------- associated items in this implementation [INFO] [stdout] ... [INFO] [stdout] 162 | pub fn set_static_private( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 442 | pub fn dst_address(packet: &[u8]) -> Option { [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 580 | pub fn is_expired(&self) -> bool { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `IncorrectPacketLength`, `WrongPacketType`, and `LockFailed` are never constructed [INFO] [stdout] --> boringtun/src/noise/errors.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 6 | pub enum WireGuardError { [INFO] [stdout] | -------------- variants in this enum [INFO] [stdout] 7 | DestinationBufferTooSmall, [INFO] [stdout] 8 | IncorrectPacketLength, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] 9 | UnexpectedPacket, [INFO] [stdout] 10 | WrongPacketType, [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 20 | LockFailed, [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `WireGuardError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `HalfHandshake` is never constructed [INFO] [stdout] --> boringtun/src/noise/handshake.rs:181:12 [INFO] [stdout] | [INFO] [stdout] 181 | pub struct HalfHandshake { [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `parse_handshake_anon` is never used [INFO] [stdout] --> boringtun/src/noise/handshake.rs:186:8 [INFO] [stdout] | [INFO] [stdout] 186 | pub fn parse_handshake_anon( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `set_static_private` is never used [INFO] [stdout] --> boringtun/src/noise/handshake.rs:248:8 [INFO] [stdout] | [INFO] [stdout] 225 | impl NoiseParams { [INFO] [stdout] | ---------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 248 | fn set_static_private( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `set_static_private` is never used [INFO] [stdout] --> boringtun/src/noise/handshake.rs:328:19 [INFO] [stdout] | [INFO] [stdout] 265 | impl Handshake { [INFO] [stdout] | -------------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 328 | pub(crate) fn set_static_private( [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: method `persistent_keepalive` is never used [INFO] [stdout] --> boringtun/src/noise/timers.rs:347:12 [INFO] [stdout] | [INFO] [stdout] 122 | impl Tunn { [INFO] [stdout] | --------- method in this implementation [INFO] [stdout] ... [INFO] [stdout] 347 | pub fn persistent_keepalive(&self) -> Option { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:17 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:24:26 [INFO] [stdout] | [INFO] [stdout] 24 | #[derive(Debug, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519SecretKey` [INFO] [stdout] 25 | /// A secret X25519 key. [INFO] [stdout] 26 | pub struct X25519SecretKey { [INFO] [stdout] | --------------- `X25519SecretKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:38 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------ [INFO] [stdout] | | [INFO] [stdout] | `Zeroize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_Zeroize_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Zeroize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Zeroize` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `Zeroize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `Drop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_Drop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> boringtun/src/crypto/x25519/mod.rs:100:47 [INFO] [stdout] | [INFO] [stdout] 100 | #[derive(Debug, PartialEq, Eq, Hash, Zeroize, ZeroizeOnDrop)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `ZeroizeOnDrop` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_zeroize_ZeroizeOnDrop_FOR_X25519PublicKey` [INFO] [stdout] 101 | /// A public X25519, derived from a secret key. [INFO] [stdout] 102 | pub struct X25519PublicKey { [INFO] [stdout] | --------------- `X25519PublicKey` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `ZeroizeOnDrop` may come from an old version of the `zeroize_derive` crate, try updating your dependency with `cargo update -p zeroize_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `ZeroizeOnDrop` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `Box::::from_raw` that must be used [INFO] [stdout] --> boringtun/src/ffi/mod.rs:236:5 [INFO] [stdout] | [INFO] [stdout] 236 | Box::from_raw(tunnel); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: call `drop(Box::from_raw(ptr))` if you intend to drop the `Box` [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 236 | let _ = Box::from_raw(tunnel); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> boringtun/src/noise/mod.rs:254:39 [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result { [INFO] [stdout] | ^^^^^ ^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 254 | pub fn parse_incoming_packet(src: &[u8]) -> Result, WireGuardError> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 13.64s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: daemonize v0.4.1 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 7` [INFO] running `Command { std: "docker" "inspect" "2523b51e7c742ce3e62d463c2d3ffd3e9edf95ce00850b52baa5f8da015f43a6", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "2523b51e7c742ce3e62d463c2d3ffd3e9edf95ce00850b52baa5f8da015f43a6", kill_on_drop: false }` [INFO] [stdout] 2523b51e7c742ce3e62d463c2d3ffd3e9edf95ce00850b52baa5f8da015f43a6