Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   =?UTF-8?q?Micha=C5=82=20Lowas-Rzechonek?= 
        <michal.lowas-rzechonek@silvair.com>
To:     linux-bluetooth@vger.kernel.org
Subject: [PATCH BlueZ] mesh: Fix ignored return value
Date:   Tue, 12 Nov 2019 15:54:25 +0100
Message-Id: <20191112145425.6318-1-michal.lowas-rzechonek@silvair.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

This fixes the following build error when compiling in maintainer mode:

mesh/keyring.c: In function ‘finalize’:
mesh/keyring.c:142:8: error: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Werror=unused-result]
  (void)write(fd, &key, sizeof(key));
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
---
 mesh/keyring.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/mesh/keyring.c b/mesh/keyring.c
index 9fa7d6bd0..fe292b4a7 100644
--- a/mesh/keyring.c
+++ b/mesh/keyring.c
@@ -140,7 +140,8 @@ static void finalize(const char *fpath, uint16_t net_idx)
 	l_debug("Finalize %s", fpath);
 	memcpy(key.old_key, key.new_key, 16);
 	lseek(fd, 0, SEEK_SET);
-	write(fd, &key, sizeof(key));
+	if (write(fd, &key, sizeof(key)) != sizeof(key))
+		goto done;
 
 done:
 	close(fd);
-- 
2.19.1