misc-string_to_version - Instead of empty string use NULL.
authorTomas Mlcoch <tmlcoch@redhat.com>
Tue, 10 Jan 2012 08:27:07 +0000 (09:27 +0100)
committerTomas Mlcoch <tmlcoch@redhat.com>
Tue, 10 Jan 2012 08:27:07 +0000 (09:27 +0100)
misc.c

diff --git a/misc.c b/misc.c
index 7fb88a3..78ea27f 100644 (file)
--- a/misc.c
+++ b/misc.c
@@ -37,12 +37,6 @@ struct VersionStruct string_to_version(const char *string)
     ver.release = NULL;
 
     if (!string || !(strlen(string))) {
-        ver.epoch = malloc(sizeof(char));
-        ver.version = malloc(sizeof(char));
-        ver.release = malloc(sizeof(char));
-        ver.epoch[0] = '\0';
-        ver.version[0] = '\0';
-        ver.release[0] = '\0';
         return ver;
     }
 
@@ -60,11 +54,11 @@ struct VersionStruct string_to_version(const char *string)
             ver.epoch = malloc(sizeof(char) * (len + 1));
             strncpy(ver.epoch, string, len);
             ver.epoch[len] = '\0';
-        } else { // epoch is not a number
-            ver.epoch = NULL;
-        }
+        } //else { // epoch is not a number
+            //ver.epoch = NULL;
+        //}
     } else {
-        ver.epoch = NULL;
+        //ver.epoch = NULL;
         ptr = string-1;
     }
 
@@ -82,18 +76,17 @@ struct VersionStruct string_to_version(const char *string)
         strcpy(ver.version, ptr+1);
     }
 
-    // Malloc empty strings instead of NULL
     if (!ver.epoch) {
         ver.epoch = malloc(sizeof(char) * 2);
         ver.epoch[0] = '0';
         ver.epoch[1] = '\0';
-    } else if (!ver.version) {
-        ver.version = malloc(sizeof(char));
-        ver.version[0] = '\0';
-    } else if (!ver.release) {
-        ver.release = malloc(sizeof(char));
-        ver.release[0] = '\0';
-    }
+    } //else if (!ver.version) {
+//        ver.version = malloc(sizeof(char));
+//        ver.version[0] = '\0';
+//    } else if (!ver.release) {
+//        ver.release = malloc(sizeof(char));
+//        ver.release[0] = '\0';
+//    }
 
     return ver;
 }