From 4cebad3859f8cc4fe526a89ef2277d1db9ee9b7c Mon Sep 17 00:00:00 2001
From: Boris Kolpackov <boris@codesynthesis.com>
Date: Tue, 6 Jul 2021 16:06:23 +0200
Subject: Clear errno before calling strto*() functions

---
 libbutl/builtin.cxx          | 1 +
 libbutl/semantic-version.cxx | 1 -
 libbutl/standard-version.cxx | 1 +
 3 files changed, 2 insertions(+), 1 deletion(-)

(limited to 'libbutl')

diff --git a/libbutl/builtin.cxx b/libbutl/builtin.cxx
index a6bb94b..61df568 100644
--- a/libbutl/builtin.cxx
+++ b/libbutl/builtin.cxx
@@ -1929,6 +1929,7 @@ namespace butl
         if (!a.empty () && a[0] != '-' && a[0] != '+')
         {
           char* e (nullptr);
+          errno = 0; // We must clear it according to POSIX.
           n = strtoull (a.c_str (), &e, 10); // Can't throw.
 
           if (errno != ERANGE && e == a.c_str () + a.size ())
diff --git a/libbutl/semantic-version.cxx b/libbutl/semantic-version.cxx
index eaf709d..445890d 100644
--- a/libbutl/semantic-version.cxx
+++ b/libbutl/semantic-version.cxx
@@ -14,7 +14,6 @@
 #include <ostream>
 
 #include <cstring>   // strchr()
-#include <cstdlib>   // strtoull()
 #include <utility>   // move()
 #include <stdexcept> // invalid_argument
 #endif
diff --git a/libbutl/standard-version.cxx b/libbutl/standard-version.cxx
index a9f5eb8..863cb29 100644
--- a/libbutl/standard-version.cxx
+++ b/libbutl/standard-version.cxx
@@ -60,6 +60,7 @@ namespace butl
 
     const char* b (s.c_str () + p);
     char* e (nullptr);
+    errno = 0; // We must clear it according to POSIX.
     uint64_t v (strtoull (b, &e, 10)); // Can't throw.
 
     if (errno == ERANGE || b == e || v < min || v > max)
-- 
cgit v1.1