@yuneta/gobj-js 7.26.28 → 7.26.29

package/dist/gobj-js.iife.js

@@ -2152,7 +2152,8 @@ var gobjJs = (function(exports) {
 		if (!gobj_is_running(gobj)) {
 			if (!(gobj.gclass.gclass_flag & gclass_flag_t.gcflag_required_start_to_play)) {
 				log_warning(`${gobj_short_name(gobj)}: GObj playing without previous start`);
-				gobj_start(gobj);
+				if (start_destroys_it(gobj)) return -1;
+				if (!gobj.running) return -1;
 			} else {
 				log_error(`${gobj_short_name(gobj)}: Cannot play, start not done`);
 				return -1;
@@ -2977,12 +2978,20 @@ var gobjJs = (function(exports) {
 	function gobj_change_parent(gobj, parent) {
 		if (gobj_is_destroying(gobj)) {
 			log_error("gobj NULL or DESTROYED");
-			return null;
+			return -1;
 		}
 		if (!(parent instanceof GObj)) {
 			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() BAD TYPE of parent`);
 			return -1;
 		}
+		if (gobj_is_destroying(parent)) {
+			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() parent NULL or DESTROYED`);
+			return -1;
+		}
+		for (let p = parent; p; p = p.parent) if (p === gobj) {
+			log_error(`${gobj_short_name(gobj)}: gobj_change_parent() a gobj cannot be its own ancestor: ${gobj_short_name(parent)}`);
+			return -1;
+		}
 		if (gobj.parent && gobj.parent.gclass.gmt.mt_child_removed) gobj.parent.gclass.gmt.mt_child_removed(gobj.parent, gobj);
 		if (gobj.parent) {
 			_remove_child(gobj.parent, gobj);
@@ -7459,9 +7468,11 @@ var gobjJs = (function(exports) {
 	*  identity card and the `host` of every message the client sends, so
 	*  the first answer is kept and answered again: 7.26.27 made a new
 	*  uuid on every call when the store could not keep it, and answered
-	*  null without crypto.randomUUID().
+	*  null without crypto.randomUUID(). A stored value that is not a
+	*  uuid (a foreign or corrupt write) is replaced, with a WARNING.
 	************************************************************/
 	var uuid_of_this_page = null;
+	var UUID_SHAPE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 	function node_uuid() {
 		if (uuid_of_this_page) return uuid_of_this_page;
 		const store = local_store();
@@ -7477,6 +7488,10 @@ var gobjJs = (function(exports) {
 			uuid_of_this_page = new_uuid();
 			return uuid_of_this_page;
 		}
+		if (uuid && !UUID_SHAPE.test(uuid)) {
+			log_warning("node_uuid(): the stored host_uuid is not a uuid: a new one replaces it");
+			uuid = null;
+		}
 		if (!uuid) {
 			uuid = new_uuid();
 			try {