upkg: Handle errors from u_object_export_name.

[upkg.git] / src / upkg.c

diff --git a/src/upkg.c b/src/upkg.c
index c13a867f7b1dfd8cbdbf21977c537ccc89d8da19..d54a5748cdd7f9d3c3c9866d9a9789557965d5c6 100644 (file)
--- a/src/upkg.c
+++ b/src/upkg.c
@@ -309,13 +309,14 @@ static int object_info(GTypeModule *pkg, unsigned long idx)
        return 0;
 }
 
-static int export(struct upkg *pkg, GObject *obj, unsigned idx)
+static int export(GTypeModule *pkg, GObject *obj, unsigned idx)
 {
-       const struct upkg_export *export = upkg_get_export(pkg, idx);
+       struct upkg *upkg = U_PKG(pkg)->pkg;
        char name[256];
        FILE *of;
+       int rc;
 
-       if (u_object_deserialize(obj, pkg, idx) != 0) {
+       if (u_object_deserialize(obj, upkg, idx) != 0) {
                return -1;
        }
 
@@ -323,9 +324,18 @@ static int export(struct upkg *pkg, GObject *obj, unsigned idx)
                return -1;
        }
 
-       u_object_export_name(obj, name, sizeof name);
+       rc = u_object_export_name(obj, name, sizeof name);
+       if (rc <= 0) {
+               /* XXX: We could use a default name here. */
+               fprintf(stderr, "%s: failed to determine export filename.\n",
+                               progname);
+               return -1;
+       }
+
+       printf("exporting ");
+       export_print_fullname(pkg, upkg_get_export(upkg, idx));
+       printf(" to %s\n", name);
 
-       printf("exporting %s to %s\n", export->name, name);
        of = fopen(name, "wb");
        if (!of) {
                perror(name);
@@ -346,15 +356,16 @@ static int export(struct upkg *pkg, GObject *obj, unsigned idx)
        return 0;
 }
 
-int package_export(struct upkg *pkg)
+int package_export(GTypeModule *pkg)
 {
+       struct upkg *upkg = U_PKG(pkg)->pkg;
        const char *class, *package;
        GObject *obj;
        GType type;
        int ret = 0;
 
-       for (unsigned i = 0; i < pkg->export_count; i++) {
-               class = upkg_export_class(pkg, i, &package);
+       for (unsigned i = 0; i < upkg->export_count; i++) {
+               class = upkg_export_class(upkg, i, &package);
                if (!class) {
                        fprintf(stderr, "error getting class information.\n");
                        return -1;
@@ -436,7 +447,7 @@ static int process_object(int mode, const char *objname)
                ret = package_list(pkg, current+1);
                break;
        case MODE_EXPORT:
-               ret = package_export(U_PKG(pkg)->pkg);
+               ret = package_export(pkg);
                break;
        default:
                abort();