From 10e3d620b46e0f28aebc858e058c52a4a63e9a91 Mon Sep 17 00:00:00 2001 From: Greg Kaiser Date: Thu, 19 Nov 2020 16:15:48 -0800 Subject: [PATCH] dmuserd: Fix error checking We change read_all() and write_all() to return signed values, so our negative error return values will be correctly caught. Test: TreeHugger Change-Id: I4d03d4475bccb40ae4b84a846966139b34ede2bd --- fs_mgr/tools/dmuserd.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs_mgr/tools/dmuserd.cpp b/fs_mgr/tools/dmuserd.cpp index 92f587860..e50a4a281 100644 --- a/fs_mgr/tools/dmuserd.cpp +++ b/fs_mgr/tools/dmuserd.cpp @@ -76,7 +76,7 @@ struct dm_user_message { static bool verbose = false; -size_t write_all(int fd, void* buf, size_t len) { +ssize_t write_all(int fd, void* buf, size_t len) { char* buf_c = (char*)buf; ssize_t total = 0; ssize_t once; @@ -94,7 +94,7 @@ size_t write_all(int fd, void* buf, size_t len) { return total; } -size_t read_all(int fd, void* buf, size_t len) { +ssize_t read_all(int fd, void* buf, size_t len) { char* buf_c = (char*)buf; ssize_t total = 0; ssize_t once;