summary refs log tree commit diff
diff options
context:
space:
mode:
authorDaniel Verkamp <dverkamp@chromium.org>2020-05-26 10:37:37 -0700
committerCommit Bot <commit-bot@chromium.org>2020-05-27 04:46:32 +0000
commit1cc1d5e6eea56c24261380fee6549b4958735d9d (patch)
treed1da535c22b09930e4a4b6556694d7574a86fb86
parentddd8180a13d0f389d533026c667c8fe57ee5e8ee (diff)
downloadcrosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar.gz
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar.bz2
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar.lz
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar.xz
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.tar.zst
crosvm-1cc1d5e6eea56c24261380fee6549b4958735d9d.zip
arch: check property_string result in create_android_fdt
Add the missing '?' to check the result of the newly-added
property_string call to match the rest in this function.

Fixes clippy warning about an unused result.

BUG=None
TEST=docker/wrapped_smoke_test.sh

Change-Id: I428b377e1a4f15e0ee96c8e96540f2fc8edce560
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2216400
Tested-by: Daniel Verkamp <dverkamp@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
Auto-Submit: Daniel Verkamp <dverkamp@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Zach Reizner <zachr@chromium.org>
Commit-Queue: Daniel Verkamp <dverkamp@chromium.org>
-rw-r--r--arch/src/android.rs2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/src/android.rs b/arch/src/android.rs
index 5311d3f..a3242ab 100644
--- a/arch/src/android.rs
+++ b/arch/src/android.rs
@@ -36,7 +36,7 @@ pub fn create_android_fdt(fdt: &mut Vec<u8>, fstab: File) -> Result<()> {
     begin_node(fdt, "vendor")?;
     for vec in dtprop {
         let content = std::fs::read_to_string(&vec[2]).map_err(Error::FdtIoError)?;
-        property_string(fdt, &vec[1], &content);
+        property_string(fdt, &vec[1], &content)?;
     }
     end_node(fdt)?; // vendor
     begin_node(fdt, "fstab")?;