Merge remote-tracking branch 'origin/master' into dev/integrate-change-column-name-fix

Author: d99kris

CMakeLists.txt

@@ -193,6 +193,7 @@ if(RAPIDCSV_BUILD_TESTS)
   endif()
   add_unit_test(test102)
   add_unit_test(test103)
+  add_unit_test(test104)
   add_unit_test(test105)
 
   # perf tests

src/rapidcsv.h

@@ -2,7 +2,7 @@
  * rapidcsv.h
  *
  * URL:      https://github.com/d99kris/rapidcsv
- * Version:  8.92
+ * Version:  8.93
  *
  * Copyright (C) 2017-2026 Kristofer Berggren
  * All rights reserved.
@@ -191,24 +191,16 @@ namespace rapidcsv
           pVal = static_cast<T>(std::stoi(pStr));
           return;
         }
-        else if (typeid(T) == typeid(signed char))
+        else if ((typeid(T) == typeid(signed char)) || (typeid(T) == typeid(unsigned char)) ||
+                 (typeid(T) == typeid(short)) || (typeid(T) == typeid(unsigned short)))
         {
-          pVal = static_cast<T>(std::stoi(pStr));
-          return;
-        }
-        else if (typeid(T) == typeid(unsigned char))
-        {
-          pVal = static_cast<T>(std::stoi(pStr));
-          return;
-        }
-        else if (typeid(T) == typeid(short))
-        {
-          pVal = static_cast<T>(std::stoi(pStr));
-          return;
-        }
-        else if (typeid(T) == typeid(unsigned short))
-        {
-          pVal = static_cast<T>(std::stoi(pStr));
+          const int i = std::stoi(pStr);
+          if ((i < static_cast<int>((std::numeric_limits<T>::min)())) ||
+              (i > static_cast<int>((std::numeric_limits<T>::max)())))
+          {
+            throw std::out_of_range("conversion: out of range");
+          }
+          pVal = static_cast<T>(i);
           return;
         }
         else if (typeid(T) == typeid(long))

tests/test104.cpp

@@ -0,0 +1,58 @@
+// test104.cpp - narrow integer type range checking
+
+#include <rapidcsv.h>
+#include "unittest.h"
+
+int main()
+{
+  int rv = 0;
+
+  std::string csv =
+    "col\n"
+    "127\n"
+    "128\n"
+    "-128\n"
+    "-129\n"
+    "255\n"
+    "256\n"
+    "32767\n"
+    "32768\n"
+    "65535\n"
+    "65536\n"
+  ;
+
+  std::string path = unittest::TempPath();
+  unittest::WriteFile(path, csv);
+
+  try
+  {
+    rapidcsv::Document doc(path);
+
+    // signed char: valid range [-128, 127]
+    unittest::ExpectEqual(int, static_cast<int>(doc.GetCell<signed char>(0, 0)), 127);
+    ExpectException(doc.GetCell<signed char>(0, 1), std::out_of_range);
+    unittest::ExpectEqual(int, static_cast<int>(doc.GetCell<signed char>(0, 2)), -128);
+    ExpectException(doc.GetCell<signed char>(0, 3), std::out_of_range);
+
+    // unsigned char: valid range [0, 255]
+    unittest::ExpectEqual(int, static_cast<int>(doc.GetCell<unsigned char>(0, 4)), 255);
+    ExpectException(doc.GetCell<unsigned char>(0, 5), std::out_of_range);
+
+    // short: valid range [-32768, 32767]
+    unittest::ExpectEqual(int, static_cast<int>(doc.GetCell<short>(0, 6)), 32767);
+    ExpectException(doc.GetCell<short>(0, 7), std::out_of_range);
+
+    // unsigned short: valid range [0, 65535]
+    unittest::ExpectEqual(int, static_cast<int>(doc.GetCell<unsigned short>(0, 8)), 65535);
+    ExpectException(doc.GetCell<unsigned short>(0, 9), std::out_of_range);
+  }
+  catch (const std::exception& ex)
+  {
+    std::cout << ex.what() << std::endl;
+    rv = 1;
+  }
+
+  unittest::DeleteFile(path);
+
+  return rv;
+}