xml_dump* modules now checks if package is NULL
authorTomas Mlcoch <tmlcoch@redhat.com>
Tue, 26 Jun 2012 11:05:55 +0000 (13:05 +0200)
committerTomas Mlcoch <tmlcoch@redhat.com>
Tue, 26 Jun 2012 11:05:55 +0000 (13:05 +0200)
src/xml_dump.c
src/xml_dump_filelists.c
src/xml_dump_other.c
src/xml_dump_primary.c

index 5309af1..2547f44 100644 (file)
@@ -85,9 +85,17 @@ void dump_files(xmlNodePtr node, Package *package, int primary)
 struct XmlStruct xml_dump(Package *pkg)
 {
     struct XmlStruct result;
-    result.primary = xml_dump_primary(pkg);
+
+    if (!pkg) {
+        result.primary   = NULL;
+        result.filelists = NULL;
+        result.other     = NULL;
+        return result;
+    }
+
+    result.primary   = xml_dump_primary(pkg);
     result.filelists = xml_dump_filelists(pkg);
-    result.other = xml_dump_other(pkg);
+    result.other     = xml_dump_other(pkg);
 
     return result;
 }
index fc24942..f68bff4 100644 (file)
@@ -76,6 +76,9 @@ void dump_filelists_items(xmlNodePtr root, Package *package)
 
 char *xml_dump_filelists(Package *package)
 {
+    if (!package)
+        return NULL;
+
     xmlNodePtr root = NULL;
     root = xmlNewNode(NULL, BAD_CAST "package");
 
index b073876..e431cd1 100644 (file)
@@ -112,6 +112,9 @@ void dump_other_items(xmlNodePtr root, Package *package)
 
 char *xml_dump_other(Package *package)
 {
+    if (!package)
+        return NULL;
+
     xmlNodePtr root = NULL;
     root = xmlNewNode(NULL, BAD_CAST "package");
 
index 6bacaa2..72d4c5b 100644 (file)
@@ -343,6 +343,9 @@ void dump_base_items(xmlNodePtr root, Package *package)
 
 char *xml_dump_primary(Package *package)
 {
+    if (!package)
+        return NULL;
+
     xmlNodePtr root = NULL;
     root = xmlNewNode(NULL, BAD_CAST "package");