From: Tomas Mlcoch <tmlcoch@redhat.com>
Date: Wed, 11 Sep 2013 14:22:45 +0000 (+0200)
Subject: xml_repomd_parser: Replace atol() with cr_xml_parser_strtoll()
X-Git-Tag: upstream/0.2.1~26
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f98f0c1c261f6ddac7432ffcff199e03f15fd561;p=services%2Fcreaterepo_c.git

xml_repomd_parser: Replace atol() with cr_xml_parser_strtoll()
---

diff --git a/src/load_metadata.c b/src/load_metadata.c
index 522863d..682d4dc 100644
--- a/src/load_metadata.c
+++ b/src/load_metadata.c
@@ -31,11 +31,6 @@
 #include "locate_metadata.h"
 #include "xml_parser.h"
 
-/** TODO:
- * - Add support for single chunk (?)
- * - Support for warning cbs
- */
-
 #define STRINGCHUNK_SIZE        16384
 
 /** Structure for loaded metadata
diff --git a/src/xml_parser_repomd.c b/src/xml_parser_repomd.c
index dcad644..f9559da 100644
--- a/src/xml_parser_repomd.c
+++ b/src/xml_parser_repomd.c
@@ -34,10 +34,6 @@
 #define ERR_DOMAIN      CR_XML_PARSER_REPOMD_ERROR
 #define ERR_CODE_XML    CRE_BADXMLREPOMD
 
-/* TODO:
- * - replace atol function with better alternative
- */
-
 typedef enum {
     STATE_START,
     STATE_REPOMD,
@@ -336,28 +332,30 @@ cr_end_handler(void *pdata, const char *element)
         assert(pd->repomd);
         assert(pd->repomdrecord);
 
-        pd->repomdrecord->timestamp = atol(pd->content);
+        pd->repomdrecord->timestamp = cr_xml_parser_strtoll(pd, pd->content, 0);
         break;
 
     case STATE_SIZE:
         assert(pd->repomd);
         assert(pd->repomdrecord);
 
-        pd->repomdrecord->size = atol(pd->content);
+        pd->repomdrecord->size = cr_xml_parser_strtoll(pd, pd->content, 0);
         break;
 
     case STATE_OPENSIZE:
         assert(pd->repomd);
         assert(pd->repomdrecord);
 
-        pd->repomdrecord->size_open = atol(pd->content);
+        pd->repomdrecord->size_open = cr_xml_parser_strtoll(pd, pd->content, 0);
         break;
 
     case STATE_DBVERSION:
         assert(pd->repomd);
         assert(pd->repomdrecord);
 
-        pd->repomdrecord->db_ver = atol(pd->content);
+        pd->repomdrecord->db_ver = (int) cr_xml_parser_strtoll(pd,
+                                                               pd->content,
+                                                               0);
         break;
 
     default: