From: Jeremy Kerr <jk@ozlabs.org>
Date: Mon, 28 Jul 2014 02:09:49 +0000 (+0800)
Subject: discover: Only update our config if the config_set succeeded.
X-Git-Tag: v1.0.0~151
X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=commitdiff_plain;h=a81a91f363acefed2f19309383877c02ff6389ef

discover: Only update our config if the config_set succeeded.

If config_set fails, we don't want to send the failed config out to
clients.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
---

diff --git a/discover/device-handler.c b/discover/device-handler.c
index e4978fa..22866e6 100644
--- a/discover/device-handler.c
+++ b/discover/device-handler.c
@@ -817,7 +817,12 @@ void device_handler_cancel_default(struct device_handler *handler)
 void device_handler_update_config(struct device_handler *handler,
 		struct config *config)
 {
-	config_set(config);
+	int rc;
+
+	rc = config_set(config);
+	if (rc)
+		return;
+
 	discover_server_notify_config(handler->server, config);
 	device_handler_reinit(handler);
 }