@yuneta/gobj-js 7.26.28 → 7.26.29

package/dist/gobj-js.es.js

@@ -2150,7 +2150,8 @@ function gobj_play(gobj) {
 	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;
@@ -2975,12 +2976,20 @@ function gobj_current_state(gobj) {
 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);
@@ -7457,9 +7466,11 @@ function new_uuid() {
 *  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();
@@ -7475,6 +7486,10 @@ function node_uuid() {
 		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 {