Tarantool development patches archive
 help / color / mirror / Atom feed
* [tarantool-patches] [PATCH 1/1] box: raise an error on nil replicaset and instance uuid
@ 2019-09-07 13:26 Vladislav Shpilevoy
  2019-09-07 13:28 ` [tarantool-patches] " Vladislav Shpilevoy
  2019-10-16  0:12 ` [Tarantool-patches] " Alexander Turenko
  0 siblings, 2 replies; 4+ messages in thread
From: Vladislav Shpilevoy @ 2019-09-07 13:26 UTC (permalink / raw)
  To: tarantool-patches; +Cc: alexander.turenko

Before the patch the nil UUID was ignored and a new random one
was generated. This was because internally box treats nil UUID
as its absense.

Now a user will see an explicit message that nil UUID is a
reserved value.

Closes #4282
---
Branch: https://github.com/tarantool/tarantool/tree/gerold103/gh-4282-box-cfg-rs-uuid
Issue: https://github.com/tarantool/tarantool/issues/4282

 src/box/box.cc            | 25 +++++++++++++++++--------
 test/app-tap/cfg.test.lua | 12 +++++++++++-
 2 files changed, 28 insertions(+), 9 deletions(-)

diff --git a/src/box/box.cc b/src/box/box.cc
index ac10c21ad..5efc88d5f 100644
--- a/src/box/box.cc
+++ b/src/box/box.cc
@@ -480,22 +480,31 @@ box_check_replication_sync_timeout(void)
 	return timeout;
 }
 
+static inline void
+box_check_uuid(struct tt_uuid *uuid, const char* name)
+{
+	*uuid = uuid_nil;
+	const char *uuid_str = cfg_gets(name);
+	if (uuid_str == NULL)
+		return;
+	if (tt_uuid_from_string(uuid_str, uuid) != 0)
+		tnt_raise(ClientError, ER_CFG, name, uuid_str);
+	if (tt_uuid_is_nil(uuid)) {
+		tnt_raise(ClientError, ER_CFG, name,
+			  tt_sprintf("nil UUID is a reserved value"));
+	}
+}
+
 static void
 box_check_instance_uuid(struct tt_uuid *uuid)
 {
-	*uuid = uuid_nil;
-	const char *uuid_str = cfg_gets("instance_uuid");
-	if (uuid_str != NULL && tt_uuid_from_string(uuid_str, uuid) != 0)
-		tnt_raise(ClientError, ER_CFG, "instance_uuid", uuid_str);
+	box_check_uuid(uuid, "instance_uuid");
 }
 
 static void
 box_check_replicaset_uuid(struct tt_uuid *uuid)
 {
-	*uuid = uuid_nil;
-	const char *uuid_str = cfg_gets("replicaset_uuid");
-	if (uuid_str != NULL && tt_uuid_from_string(uuid_str, uuid) != 0)
-		tnt_raise(ClientError, ER_CFG, "replicaset_uuid", uuid_str);
+	box_check_uuid(uuid, "replicaset_uuid");
 }
 
 static enum wal_mode
diff --git a/test/app-tap/cfg.test.lua b/test/app-tap/cfg.test.lua
index 7d8e9a05e..67af00c30 100755
--- a/test/app-tap/cfg.test.lua
+++ b/test/app-tap/cfg.test.lua
@@ -3,7 +3,17 @@ local fiber = require('fiber')
 local tap = require('tap')
 local test = tap.test("cfg")
 
-test:plan(8)
+test:plan(10)
+
+--
+-- gh-4282: box.cfg should not allow nor just ignore nil UUID.
+-- It is a special reserved value.
+--
+local nil_uuid = '00000000-0000-0000-0000-000000000000'
+local ok = pcall(box.cfg, {instance_uuid = nil_uuid})
+test:ok(not ok, 'nil instance UUID is not allowed')
+ok, err = pcall(box.cfg, {replicaset_uuid = nil_uuid})
+test:ok(not ok, 'nil replicaset UUID is not allowed')
 
 test:is(type(box.ctl), "table", "box.ctl is available before box.cfg")
 test:is(type(box.ctl.wait_ro), "function", "box.ctl.wait_ro is available")
-- 
2.20.1 (Apple Git-117)

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-10-16  0:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-07 13:26 [tarantool-patches] [PATCH 1/1] box: raise an error on nil replicaset and instance uuid Vladislav Shpilevoy
2019-09-07 13:28 ` [tarantool-patches] " Vladislav Shpilevoy
2019-09-08 12:27   ` Vladislav Shpilevoy
2019-10-16  0:12 ` [Tarantool-patches] " Alexander Turenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox