X-Git-Url: https://git.draconx.ca/gitweb/upkg.git/blobdiff_plain/e3726c0480d74840dd9104850cba0f3e5f93beed..8a919f09087cf70e15196f79aa8e885a8fa01595:/src/uobject/uobject.c

diff --git a/src/uobject/uobject.c b/src/uobject/uobject.c
index 1a3d740..b8c2479 100644
--- a/src/uobject/uobject.c
+++ b/src/uobject/uobject.c
@@ -27,6 +27,7 @@
 
 #include <uobject/uobject.h>
 #include <uobject/module.h>
+#include <uobject/package.h>
 #include "upkg.h"
 #include "pack.h"
 
@@ -174,7 +175,7 @@ decode_prop_header(struct upkg *upkg, struct prop_head *head,
 	}
 
 	rc = decode_tag_size(&head->size, tag_size, buf+len, n-len);
-	if (head->size == 0)
+	if (rc == 0 && head->size == 0)
 		return 0;
 	len += rc;
 
@@ -217,7 +218,7 @@ static unsigned long deserialize_property(UObject *uo, struct prop_head *head)
 	unsigned long rc, len = 0;
 	GValue val = {0};
 
-	rc = decode_prop_header(uo->pkg, head, priv->buf, priv->nbuf);
+	rc = decode_prop_header(uo->pkg_file->pkg, head, priv->buf, priv->nbuf);
 	if (rc == 0)
 		return 0;
 	len += rc;
@@ -239,6 +240,11 @@ static unsigned long deserialize_property(UObject *uo, struct prop_head *head)
 		g_value_set_ulong(&val, unpack_32_le(priv->buf+len));
 		g_object_set_property(G_OBJECT(uo), head->prop_name, &val);
 		break;
+	case PROPERTY_BOOLEAN:
+		g_value_init(&val, G_TYPE_BOOLEAN);
+		g_value_set_boolean(&val, head->tag_msb);
+		g_object_set_property(G_OBJECT(uo), head->prop_name, &val);
+		break;
 	case PROPERTY_OBJECT:
 		rc = decode_object_property(uo, &val, len);
 		if (rc != 0)
@@ -306,22 +312,73 @@ static int deserialize(UObject *uo)
 	return 0;
 }
 
-int u_object_deserialize(GObject *obj, struct upkg *pkg, unsigned long idx)
+/*
+ * Get the full hierarchical object name for an export, used for diagnostics.
+ * The package name is passed in pname.
+ *
+ * Returns a buffer allocated by malloc on success, or NULL on failure.
+ */
+static char *get_obj_fullname(const char *pname, const struct upkg_export *e)
+{
+	size_t total_len = strlen(pname) + 1, len;
+	char *fullname;
+
+	assert(e != NULL);
+
+	for (const struct upkg_export *c = e; c; c = c->parent) {
+		len = strlen(c->name) + 1;
+		if (total_len > SIZE_MAX - len)
+			return NULL;
+		total_len += len;
+	}
+
+	fullname = malloc(total_len);
+	if (!fullname)
+		return NULL;
+
+	for (const struct upkg_export *c = e; c; c = c->parent) {
+		len = strlen(c->name);
+		assert(total_len > len);
+
+		total_len -= len + 1;
+		memcpy(fullname + total_len, c->name, len);
+		fullname[total_len + len] = c == e ? '\0' : '.';
+	}
+
+	assert(total_len == strlen(pname)+1);
+	memcpy(fullname, pname, total_len-1);
+	fullname[total_len-1] = '.';
+
+	return fullname;
+}
+
+int u_object_deserialize(GObject *obj, GTypeModule *pkg, unsigned long idx)
 {
 	g_return_val_if_fail(IS_U_OBJECT(obj), -1);
+	g_return_val_if_fail(IS_U_PKG(pkg), -1);
+
 	UObject *uo = U_OBJECT(obj);
+	struct upkg *upkg = U_PKG(pkg)->pkg;
+	const struct upkg_export *e;
 	struct upkg_file *f;
 	int rc;
 
 	g_return_val_if_fail(uo->pkg_file == NULL, -1);
-	f = upkg_export_open(pkg, idx);
+	f = upkg_export_open(upkg, idx);
 	if (!f) {
 		return -1;
 	}
 
-	uo->pkg      = pkg;
+	uo->pkg = pkg;
 	uo->pkg_file = f;
 
+	e = upkg_get_export(upkg, idx);
+	if (!e)
+		return -1;
+	uo->pkg_name = get_obj_fullname(pkg->name, e);
+	if (!uo->pkg_name)
+		return -1;
+
 	rc = U_OBJECT_GET_CLASS(obj)->deserialize(uo);
 	if (rc != 0) {
 		upkg_export_close(f);
@@ -331,19 +388,21 @@ int u_object_deserialize(GObject *obj, struct upkg *pkg, unsigned long idx)
 	return rc;
 }
 
-GObject *u_object_new_from_package(struct upkg *upkg, unsigned long idx)
+GObject *u_object_new_from_package(GTypeModule *pkg, unsigned long idx)
 {
+	g_return_val_if_fail(IS_U_PKG(pkg), NULL);
+
 	const struct upkg_export *export;
 	const char *class, *package;
 	GObject *obj = NULL;
 	GType type;
 
-	class = upkg_export_class(upkg, idx, &package);
+	class = upkg_export_class(U_PKG(pkg)->pkg, idx, &package);
 
 	type = u_object_module_get_class(package, class);
 	if (type) {
 		obj = g_object_new(type, NULL);
-		if (u_object_deserialize(obj, upkg, idx) != 0) {
+		if (u_object_deserialize(obj, pkg, idx) != 0) {
 			g_object_unref(obj);
 			return NULL;
 		}
@@ -366,6 +425,8 @@ static void u_object_finalize(GObject *o)
 		upkg_export_close(uo->pkg_file);
 	}
 
+	free(uo->pkg_name);
+
 	G_OBJECT_CLASS(u_object_parent_class)->finalize(o);
 }
 
@@ -422,12 +483,8 @@ void u_vlog_full(GObject *o, GLogLevelFlags level, const char *fmt, va_list ap)
 	UObject *uo = U_OBJECT(o);
 	char *new_fmt = NULL;
 
-	if (uo->pkg_file) {
-		/* XXX: Currently, there's no way to get the full object name
-		 * here because the object path information is discarded
-		 * after opening it.  In principle, there's no reason why
-		 * we couldn't keep it around to improve log messages. */
-		new_fmt = prepend_fmt(uo->pkg_file->name, fmt);
+	if (uo->pkg_name) {
+		new_fmt = prepend_fmt(uo->pkg_name, fmt);
 		if (!new_fmt) {
 			g_log(G_OBJECT_TYPE_NAME(o), level, "%s",
 			      uo->pkg_file->name);